Skip to content

[DO NOT MERGE] restore broken branch for regression test isolation#9711

Closed
TomAFrench wants to merge 1 commit intomasterfrom
revert-9710-ab/revert-9659
Closed

[DO NOT MERGE] restore broken branch for regression test isolation#9711
TomAFrench wants to merge 1 commit intomasterfrom
revert-9710-ab/revert-9659

Conversation

@TomAFrench
Copy link
Member

@TomAFrench TomAFrench commented Sep 2, 2025

Reverts #9710

This PR restores the broken state prior to #9710 so that we can find a regression test for the issue which that PR fixes. cc @asterite

@TomAFrench
Copy link
Member Author

aztec-packages branch: AztecProtocol/aztec-packages#16714

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2025

Changes to number of Brillig opcodes executed

Generated at commit: 7e67aecd7d36641ed54b6926689a5d7700389d05, compared to commit: 2289e0715c7b2c5e45a9123e28605ef7db101f75

🧾 Summary (10% most significant diffs)

Program Brillig opcodes (+/-) %
regression_8739_inliner_min -40 ✅ -68.97%
regression_8739_inliner_zero -40 ✅ -68.97%

Full diff report 👇
Program Brillig opcodes (+/-) %
hashmap_inliner_min 79,994 (+48) +0.06%
uhashmap_inliner_min 174,388 (+48) +0.03%
references_inliner_min 446 (-11) -2.41%
references_inliner_zero 315 (-11) -3.37%
reference_only_used_as_alias_inliner_min 237 (-13) -5.20%
assign_ex_inliner_min 28 (-11) -28.21%
assign_ex_inliner_zero 28 (-11) -28.21%
closures_mut_ref_inliner_min 23 (-11) -32.35%
closures_mut_ref_inliner_zero 23 (-11) -32.35%
regression_8739_inliner_min 18 (-40) -68.97%
regression_8739_inliner_zero 18 (-40) -68.97%

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2025

Changes to Brillig bytecode sizes

Generated at commit: 7e67aecd7d36641ed54b6926689a5d7700389d05, compared to commit: 2289e0715c7b2c5e45a9123e28605ef7db101f75

🧾 Summary (10% most significant diffs)

Program Brillig opcodes (+/-) %
closures_mut_ref_inliner_min -7 ✅ -20.59%
closures_mut_ref_inliner_zero -7 ✅ -20.59%
regression_8739_inliner_min -39 ✅ -66.10%
regression_8739_inliner_zero -39 ✅ -66.10%

Full diff report 👇
Program Brillig opcodes (+/-) %
uhashmap_inliner_min 7,289 (-136) -1.83%
array_sort_inliner_min 461 (-9) -1.91%
hashmap_inliner_min 9,133 (-192) -2.06%
references_inliner_min 279 (-7) -2.45%
references_inliner_zero 208 (-7) -3.26%
reference_only_used_as_alias_inliner_min 214 (-9) -4.04%
assign_ex_inliner_min 34 (-7) -17.07%
assign_ex_inliner_zero 34 (-7) -17.07%
closures_mut_ref_inliner_min 27 (-7) -20.59%
closures_mut_ref_inliner_zero 27 (-7) -20.59%
regression_8739_inliner_min 20 (-39) -66.10%
regression_8739_inliner_zero 20 (-39) -66.10%

@TomAFrench TomAFrench closed this Nov 19, 2025
@TomAFrench TomAFrench deleted the revert-9710-ab/revert-9659 branch January 27, 2026 14:02
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.

1 participant