continued support for noir -> r1cs#1
Conversation
dolores-park
commented
Feb 26, 2025
- add r1cs optimization, 1 poseidon row count reduced from 644->428
| let term = expr.mul_terms[0]; | ||
| a = vec![(FieldElement::one(), self.map_witness(term.1))]; | ||
| b = vec![(FieldElement::one(), self.map_witness(term.2))]; | ||
| } |
There was a problem hiding this comment.
There is a more general result: The last mul term never needs an intermediate constraint (even if there are multiple mul-terms).
|
fixed witness generation |
|
@recmo think this is a good place to merge into main, would love to get feedback / approval from you
to test, On a separate note, as a blackbox function anyways, so will try to |
|
Added some scripts from @RyanCao7 , to run use see the outputs at |
Add Brillig support
|
To have a full end to end run, after having the noir intermediate representations, compile file at after that, there should be a ========================================================================================== copy that to the repo at checkout the repo at ========================================================================================== |
|
For above instructions to work it is important that the noirup -C 03b58fa2 |
continued support for noir -> r1cs