AztecProtocol / barretenberg

Apache License 2.0
129 stars 78 forks source link

Optimize mulitithread MSM builder row creation #973

Open codygunton opened 2 months ago

codygunton commented 2 months ago

The multithreading of the MSM builder was the culprit of sporadic failure of the test GoblinRecursionTests.Vanilla, so it was temporarily disabled. This lead to a small regression in the benchmark. If we ever get proof construction times down dramatically to where .2s matters, we should multithread this part of the code again.