From 6c7062443ae23cc75ac06b7ac1492d12f803d0e5 Mon Sep 17 00:00:00 2001 From: Cody Gunton Date: Wed, 13 Mar 2024 10:25:09 -0400 Subject: [PATCH] fix: Intermittent invert 0 in Goblin (#5189) https://github.com/AztecProtocol/aztec-packages/pull/5174 provided a general fix for this issue, but did not implement it in the particular case of the translator test, leading to a regression. We fix that. --- .../translator_vm/goblin_translator_composer.test.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/barretenberg/cpp/src/barretenberg/translator_vm/goblin_translator_composer.test.cpp b/barretenberg/cpp/src/barretenberg/translator_vm/goblin_translator_composer.test.cpp index 0bd890ccd34..7e2d37aba96 100644 --- a/barretenberg/cpp/src/barretenberg/translator_vm/goblin_translator_composer.test.cpp +++ b/barretenberg/cpp/src/barretenberg/translator_vm/goblin_translator_composer.test.cpp @@ -55,6 +55,8 @@ TEST_F(GoblinTranslatorComposerTests, Basic) // Add the same operations to the ECC op queue; the native computation is performed under the hood. auto op_queue = std::make_shared(); + op_queue->append_nonzero_ops(); + for (size_t i = 0; i < 500; i++) { op_queue->add_accumulate(P1); op_queue->mul_accumulate(P2, z);