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

Remove todo to generate a witness when resolving a NodeId #758

Closed
Tracked by #1376
kevaundray opened this issue Feb 6, 2023 · 1 comment
Closed
Tracked by #1376

Remove todo to generate a witness when resolving a NodeId #758

kevaundray opened this issue Feb 6, 2023 · 1 comment
Assignees
@guipublic
Labels
refactor ssa

Comments

@kevaundray
Copy link
Contributor

See this line:

noir/crates/noirc_evaluator/src/ssa/acir_gen.rs

Line 608 in b8c582f

todo!("generate a witness");

kevaundray added a commit that referenced this issue Feb 7, 2023
@kevaundray
remove TODO see issue #758
f716a41
@TomAFrench
Copy link
Member

There's been some churn on where this todo lives however it still exists

@Savio-Sou Savio-Sou mentioned this issue May 19, 2023
Closed
@kevaundray kevaundray closed this as not planned Won't fix, can't repro, duplicate, stale Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guipublic guipublic

Labels
refactor ssa
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@TomAFrench @kevaundray @guipublic