From 2fbceb37cffd2eb9317f330ae9a108f452e5d4f9 Mon Sep 17 00:00:00 2001 From: Innokentii Sennovskii Date: Wed, 3 May 2023 19:13:02 +0100 Subject: [PATCH] Small change that was left out of construct_addition_chains fix (#404) Fixes the intermittent construct_addition_chains bugs (the previous fix was incomplete) and cleans the test up a bit --- .../scalar_multiplication/scalar_multiplication.test.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cpp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp b/cpp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp index a09245c1e7..bd0875ba96 100644 --- a/cpp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp +++ b/cpp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp @@ -482,15 +482,15 @@ TEST(scalar_multiplication, construct_addition_chains) bucket_counts, &bit_offsets[0], state.point_schedule, - num_points, + static_cast(state.round_counts[0]), static_cast(num_buckets), bucket_empty_status }; start = std::chrono::steady_clock::now(); scalar_multiplication::construct_addition_chains(product_state, true); - // scalar_multiplication::scalar_multiplication_internal(state, monomials); end = std::chrono::steady_clock::now(); diff = std::chrono::duration_cast(end - start); + info("construct addition chains: ", diff.count(), "ms"); std::cout << "scalar mul: " << diff.count() << "ms" << std::endl; aligned_free(bucket_empty_status);