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.

Sign up for GitHub

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

Jump to bottom

Add command to dump ACIR #6

Closed
kevaundray opened this issue Mar 15, 2021 · 4 comments · Fixed by #1328
Closed

Add command to dump ACIR #6

kevaundray opened this issue Mar 15, 2021 · 4 comments · Fixed by #1328
Labels
nargo Noir's CLI development tool

Comments

@kevaundray
Copy link
Contributor

Taken from a conversation with Kobi Gurkan
@kevaundray kevaundray added the nargo Noir's CLI development tool label Mar 15, 2021
guipublic added a commit to guipublic/noir that referenced this issue Nov 25, 2021
@guipublic
installing barretenberg dependencies noir-lang#6
d7fcf9f
@kevaundray
Copy link
Contributor Author

@joss-aztec can this be closed?

@joss-aztec
Copy link
Contributor

joss-aztec commented May 9, 2023 •
edited
Loading

Probably yes for their purposes. Arguably the command is incomplete because it is doesn't work with the --contracts flag, and therefore only supports regular noir programs.

@joss-aztec
Copy link
Contributor

joss-aztec commented May 9, 2023 •
edited
Loading

I'm tempted to retire the nargo print-acir command in favour of a nargo compile --show-acir flag. That would be more versatile and result in less duplication. This may seem redundant as the --show-ssa flag already has this effect, but arguably it should be a distinct capability (i.e. the user could choose to show one, the other, both, or neither.)

@joss-aztec joss-aztec mentioned this issue May 9, 2023
Merged
6 tasks
@joss-aztec
Copy link
Contributor

(Changed mind again - see PR: #1328)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
nargo Noir's CLI development tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(nargo)!: retire print-acir in favour of flag noir-lang/noir
2 participants
@kevaundray @joss-aztec