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
With the move to non-inlined ACIR calls we need to appropriately account for a program having multiple returns. Initial work for non-inlining ACIR was started here (AztecProtocol/aztec-packages#5341), however, it is missing recursive calls.
Happy Case
I should be able to recurse non-inlined ACIR functions.
Something like this should be feasible:
fnmain(x:u32){assert(fibonacci(x) == 8);}#[fold]fnfibonacci(x:u32) -> u32{if x <= 1{
x
}else{fibonacci(x - 1) + fibonacci(x - 2)}}
Project Impact
None
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
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:
# Description
## Problem\*
Resolves#4616
## Summary\*
I wrote the initial issue for recursive calls a while ago. I thought I
had an error at the time and that is why I wrote the issue but it looks
to all be working for fibonacci example. This simply adds a test to
validate recursive ACIR calls.
## Additional Context
## Documentation\*
Check one:
- [ ] No documentation needed.
- [ ] Documentation included in this PR.
- [X] **[For Experimental Features]** Documentation to be submitted in a
separate PR.
# PR Checklist\*
- [X] I have tested the changes locally.
- [X] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
Problem
With the move to non-inlined ACIR calls we need to appropriately account for a program having multiple returns. Initial work for non-inlining ACIR was started here (AztecProtocol/aztec-packages#5341), however, it is missing recursive calls.
Happy Case
I should be able to recurse non-inlined ACIR functions.
Something like this should be feasible:
Project Impact
None
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
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: