search

AztecProtocol / barretenberg

Apache License 2.0
133 stars 81 forks source link

Sumcheck loop ordering #728

Open lucasxia01 opened 1 year ago

lucasxia01 commented 1 year ago

Brought up in a discussion with Zac over potential slowdowns in Sumcheck.

Currently we iterate over each polynomial and for each one, we call add_scaled, which loops over the same batched_poly_unshifted and batched_poly_to_be_shifted memory over and over again. Instead, we should consider switching the order of the loops so that we iterate over the polynomial indices in the outer loop and over the polynomials in the inner loop.

ledwards2225 commented 5 months ago

@lucasxia01 Is this still relevant? If so, can you clarify what part of the code this refers to?