-
Notifications
You must be signed in to change notification settings - Fork 219
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test closures_mut_ref
fails with inliner <= 0 and brillig
#6440
Labels
bug
Something isn't working
Comments
aakoshh
changed the title
closures_mut_ref fails with inliner <= 0 and brillig
Test closures_mut_ref fails with inliner <= 0 and brillig
Nov 4, 2024
aakoshh
changed the title
Test closures_mut_ref fails with inliner <= 0 and brillig
Test Nov 4, 2024
closures_mut_ref
fails with inliner <= 0 and brillig
This was referenced Nov 4, 2024
The problem indeed seems to be the same as #6443 Reduced the test program to be just the following:
The test fails at the assertion. Looking at the SSA, we can see that the ❯ cargo run -q -p nargo_cli -- --program-dir . execute --force --force-brillig --inliner-aggressiveness -9223372036854775808 --show-ssa
Initial SSA:
brillig(inline) fn main f0 {
b0(v0: Field):
v1 = allocate
store v0 at v1
call f1()
return
}
acir(inline) fn issue_2120 f1 {
b0():
v0 = allocate
store Field 42 at v0
call f2(v0, Field 44)
v4 = load v0
v5 = eq v4, Field 44
constrain v4 == Field 44
return
}
acir(inline) fn lambda f2 {
b0(v0: &mut Field, v1: Field):
v2 = allocate
store v0 at v2
v3 = load v2
store v1 at v3
return
}
...
After Array Set Optimizations:
brillig(inline) fn main f0 {
b0(v0: Field):
call f1()
return
}
brillig(inline) fn issue_2120 f1 {
b0():
v0 = allocate
store Field 42 at v0
call f2(v0, Field 44)
v4 = load v0
constrain v4 == Field 44
return
}
brillig(inline) fn lambda f2 {
b0(v0: &mut Field, v1: Field):
return
} |
Fixed by #6452 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Aim
Trying to run
test_programs
with different--inliner-aggressiveness
in #6429Expected Behavior
Tests under
execution_success
should work.Bug
To Reproduce
cd test_programs/execution_success/closures_mut_ref
nargo execute --force --force-brillig --inliner-aggressiveness 0
ornargo execute --force --force-brillig --inliner-aggressiveness -9223372036854775808
Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Nargo Version
nargo version = 0.36.0 noirc version = 0.36.0+2f0cb3e80f3d93a1dee77fffacc397811e300257 (git version hash: 2f0cb3e, is dirty: false)
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: