-
Notifications
You must be signed in to change notification settings - Fork 266
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
chore: update ACIR serialisation format #2771
Conversation
7d60868
to
6f03f4f
Compare
thinking aloud: Do we update this in tandem with a noir tag bump? |
What does the noir tag feed into in this repository? We don't have a fully working noir tag for this serialisation yet (as we rely on a compatible release of |
Because we recently added integrations tests which effectively test barretenberg again (we should remove these) it makes this process a bit tedious |
Benchmark resultsAll benchmarks are run on txs on the This benchmark source data is available in JSON format on S3 here. Values are compared against data from master at commit L2 block published to L1Each column represents the number of txs on an L2 block published to L1.
L2 chain processingEach column represents the number of blocks on the L2 chain where each block has 16 txs.
Circuits statsStats on running time and I/O sizes collected for every circuit run across all benchmarks.
|
This PR updates the ACIR serialization format to match that defined in noir-lang/noir#2872. The main change is that
Brillig
ACIR opcodes no longer contain aforeign_call_results
field.Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.