You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Attempted to benchmark that the number of executed Brillig opcodes was the same across nargo versions that produce identical SSA.
Expected Behavior
Expected that the number of opcodes would be the same or else that we have made improvements to the number of opcodes executed.
Bug
In #6981, I found a case where the final SSA is identical between two versions, but we're not using the final SSA in order to generate the Brillig that ends up in the build artifact.
This appears to be since the following two passes are being run on Brillig functions in optimize_into_acir while assuming that they'll only run on ACIR:
If you look at the SSA printed out by nargo compile --force-brillig --inliner-aggressiveness 0 --show-ssa-pass "Array Set Optimizations" then you'll see:
Aim
Attempted to benchmark that the number of executed Brillig opcodes was the same across
nargo
versions that produce identical SSA.Expected Behavior
Expected that the number of opcodes would be the same or else that we have made improvements to the number of opcodes executed.
Bug
In #6981, I found a case where the final SSA is identical between two versions, but we're not using the final SSA in order to generate the Brillig that ends up in the build artifact.
This appears to be since the following two passes are being run on Brillig functions in
optimize_into_acir
while assuming that they'll only run on ACIR:If you look at the SSA printed out by
nargo compile --force-brillig --inliner-aggressiveness 0 --show-ssa-pass "Array Set Optimizations"
then you'll see:and we then get an extra function
To Reproduce
Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Nargo Version
No response
NoirJS Version
No response
Proving Backend Tooling & Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered: