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

feat: Sync from aztec-packages #6824

Merged
merged 5 commits into from
Dec 16, 2024
Merged

feat: Sync from aztec-packages #6824

merged 5 commits into from
Dec 16, 2024

Conversation

AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Dec 16, 2024

Automated pull of Noir development from aztec-packages.
BEGIN_COMMIT_OVERRIDE
END_COMMIT_OVERRIDE

Copy link
Contributor

Changes to Brillig bytecode sizes

Generated at commit: 1ad3face7f57f3ed3a4becd6c1238e6dd5ef9ceb, compared to commit: f337992de96ef656681ebfc96a30c2c9c9b82a70

🧾 Summary (10% most significant diffs)

Program Brillig opcodes (+/-) %
poseidonsponge_x5_254 -3 ✅ -0.07%

Full diff report 👇
Program Brillig opcodes (+/-) %
poseidonsponge_x5_254 4,251 (-3) -0.07%

Copy link
Contributor

Peak Memory Sample

Program Peak Memory %
keccak256 77.76M 0%
workspace 121.97M -1%
regression_4709 273.24M 0%
ram_blowup_regression 1.60G 0%
private-kernel-tail 204.31M 0%
private-kernel-reset 746.63M 0%
private-kernel-inner 310.59M 0%
parity-root 169.56M 0%

Copy link
Contributor

Compilation Sample

Program Compilation Time %
sha256_regression 0m1.376s -1%
regression_4709 0m0.703s -6%
ram_blowup_regression 0m14.790s -1%
rollup-base-public 3m16.373s -4%
rollup-base-private 3m8.148s 4%
private-kernel-tail 0m1.389s 12%
private-kernel-reset 0m8.233s -1%
private-kernel-inner 0m2.548s 7%
parity-root 0m1.134s 37%
noir-contracts 2m54.294s 2%

@TomAFrench TomAFrench added this pull request to the merge queue Dec 16, 2024
Merged via the queue into master with commit b3bca76 Dec 16, 2024
66 checks passed
@TomAFrench TomAFrench deleted the aztec-packages branch December 16, 2024 18:21
AztecBot added a commit to AztecProtocol/aztec-packages that referenced this pull request Dec 17, 2024
…eter (noir-lang/noir#6829)

chore: Use Vec for callstacks (noir-lang/noir#6821)
feat: replace `eval_global_as_array_length` with type/interpreter evaluation (noir-lang/noir#6469)
chore: refactor `DataFlowGraph.insert_instruction_and_results` (noir-lang/noir#6823)
chore(docs): updating noirjs tutorial for 1.0.0 (noir-lang/noir#6792)
feat: Sync from aztec-packages (noir-lang/noir#6824)
chore: Have rust-analyzer use the stable toolchain (noir-lang/noir#6825)
AztecBot added a commit to AztecProtocol/aztec-packages that referenced this pull request Dec 18, 2024
…r#6810)

chore: move constant creation out of loop (noir-lang/noir#6836)
fix: implement `as_field` and `from_field` in the interpreter (noir-lang/noir#6829)
chore: Use Vec for callstacks (noir-lang/noir#6821)
feat: replace `eval_global_as_array_length` with type/interpreter evaluation (noir-lang/noir#6469)
chore: refactor `DataFlowGraph.insert_instruction_and_results` (noir-lang/noir#6823)
chore(docs): updating noirjs tutorial for 1.0.0 (noir-lang/noir#6792)
feat: Sync from aztec-packages (noir-lang/noir#6824)
chore: Have rust-analyzer use the stable toolchain (noir-lang/noir#6825)
AztecBot added a commit to AztecProtocol/aztec-packages that referenced this pull request Dec 18, 2024
chore: move constant creation out of loop (noir-lang/noir#6836)
fix: implement `as_field` and `from_field` in the interpreter (noir-lang/noir#6829)
chore: Use Vec for callstacks (noir-lang/noir#6821)
feat: replace `eval_global_as_array_length` with type/interpreter evaluation (noir-lang/noir#6469)
chore: refactor `DataFlowGraph.insert_instruction_and_results` (noir-lang/noir#6823)
chore(docs): updating noirjs tutorial for 1.0.0 (noir-lang/noir#6792)
feat: Sync from aztec-packages (noir-lang/noir#6824)
chore: Have rust-analyzer use the stable toolchain (noir-lang/noir#6825)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants