Skip to content
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

Generate RecursiveAggregation AcirOpcode in SSA refactor #1570

Closed
Tracked by #28
vezenovm opened this issue Jun 6, 2023 · 2 comments · Fixed by #2097
Closed
Tracked by #28

Generate RecursiveAggregation AcirOpcode in SSA refactor #1570

vezenovm opened this issue Jun 6, 2023 · 2 comments · Fixed by #2097
Assignees
Labels
enhancement New feature or request

Comments

@vezenovm
Copy link
Contributor

vezenovm commented Jun 6, 2023

Problem

The SSA refactor currently does not handle calling the RecursiveAggregation black box func. This should be included to enable compilation of verify_proof to match the current SSA pass.

Happy Case

We should be able to successfully generate a BlackBoxFuncCall::RecursiveAggregation ACIR opcode that can then be accepted by a prover (currently only bb.js to generate recursive proofs).

Alternatives Considered

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

@vezenovm vezenovm added the enhancement New feature or request label Jun 6, 2023
@vezenovm vezenovm self-assigned this Jun 6, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Jun 6, 2023
@kevaundray
Copy link
Contributor

@vezenovm what is the status of this?

@vezenovm
Copy link
Contributor Author

@kevaundray still todo. I will add this when I'm back next week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
2 participants