Skip to content

fix[venom]: fix clean_stack_from_cfg_in for incorrect dead items in stack - #5253

Merged
harkal merged 6 commits into
vyperlang:masterfrom
HodanPlodky:fix/venom/venom_to_asm_liveness_dependace_possible_compile_panic
Sep 21, 2026
Merged

harkal merged 6 commits into
vyperlang:masterfrom
HodanPlodky:fix/venom/venom_to_asm_liveness_dependace_possible_compile_panic

Conversation

@HodanPlodky

@HodanPlodky HodanPlodky commented Sep 9, 2026 •

Copy link
Copy Markdown
Collaborator

What I did

Implementation of the fix that was shown by the #5219.

There was dependency on liveness in the clean_stack_from_cfg_in. For the offset and phi instructions the output could have been in the stack even if was never used.

How I did it

How to verify it

Commit message

Remove the dependency on liveness in  clean_stack_from_cfg_in function which could create a CompilerPanic

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Gas Changes

No changes detected.

Summary

  • Total tests measured: 560
  • Changed: 0
  • Regressions (gas up): 0
  • Improvements (gas down): 0
  • New tests: 0
  • Deleted tests: 0
  • Newly failing: 0
  • Newly passing: 0

@github-actions

github-actions Bot commented Sep 9, 2026 •

Copy link
Copy Markdown

📊 Bytecode Size Changes (venom)

No changes detected.

Full bytecode sizes

Contract legacy-O2 legacy-Os -O2 -O3 -Os
curvefi/legacy/CurveStableSwapMetaNG.vy 24941 23567 19769 19059 18523
curvefi/amm/stableswap/meta_implementation/meta_implementation_v_700.vy 23599 22794 19521 18627 18308
curvefi/amm/stableswap/implementation/implementation_v_700.vy 24951 23758 19139 18363 18013
curvefi/legacy/CurveStableSwapNG.vy 24462 23287 18701 17981 17631
curvefi/amm/tricryptoswap/implementation/implementation_v_200.vy 20724 19959 17175 16689 16325
curvefi/amm/twocryptoswap/implementation/implementation_v_210.vy 17634 16894 14901 14376 14027
yearnfi/VaultV3.vy 19972 19063 14531 13818 13247
curvefi/legacy/CurveCryptoSwap2.vy 18947 18382 14474 14129 13935
yearnfi/VaultV2.vy 16676 15763 13211 12466 12048
curvefi/amm/stableswap/factory/factory_v_100.vy 14558 13978 11724 10780 10855
curvefi/gauge/child_gauge/implementation/implementation_v_110.vy 12338 11561 9774 9184 8795
curvefi/amm/stableswap/views/views_v_120.vy 12784 12368 9663 9059 9294
curvefi/gauge/child_gauge/implementation/implementation_v_100.vy 12017 11249 9507 8924 8538
curvefi/amm/tricryptoswap/math/math_v_200.vy 11189 11126 8923 8144 8150
curvefi/legacy/CurveCryptoMathOptimized3.vy 11188 11125 8922 8144 8150
curvefi/gauge/child_gauge/implementation/implementation_v_020.vy 10665 9947 8606 8100 7714
curvefi/registries/metaregistry/metaregistry_v_110.vy 7590 6732 6481 5710 5603
curvefi/helpers/router/router_v_110.vy 6717 6717 6160 5733 6031
curvefi/amm/tricryptoswap/views/views_v_200.vy 7821 7776 6091 5896 6037
curvefi/helpers/stable_swap_meta_zap/stable_swap_meta_zap_v_100.vy 7302 7067 5877 5350 5610
curvefi/amm/twocryptoswap/views/views_v_200.vy 6991 6946 5660 5479 5606
curvefi/registries/metaregistry/registry_handlers/stableswap/handler_v_110.vy 6633 6259 5533 4695 5238
curvefi/amm/twocryptoswap/math/math_v_210.vy 6800 6800 5501 5012 5039
curvefi/amm/twocryptoswap/factory/factory_v_200.vy 5540 5252 4480 3917 4015
curvefi/amm/tricryptoswap/factory/factory_v_200.vy 5246 5021 4377 3890 3996
curvefi/gauge/child_gauge/factory/factory_v_201.vy 4844 4547 3895 3675 3511
curvefi/registries/metaregistry/registry_handlers/tricryptoswap/handler_v_110.vy 4241 3939 3718 3334 3429
curvefi/registries/metaregistry/registry_handlers/twocryptoswap/handler_v_110.vy 4186 3884 3671 3251 3329
curvefi/gauge/child_gauge/factory/factory_v_100.vy 4183 3914 3408 3144 2971
yearnfi/VaultFactory.vy 3765 3617 2713 2158 2434
curvefi/registries/address_provider/address_provider_v_201.vy 2973 2782 2613 2440 2353
curvefi/helpers/rate_provider/rate_provider_v_101.vy 3260 3260 2535 2263 2296
curvefi/amm/stableswap/math/math_v_100.vy 3067 3046 2453 2253 2310
curvefi/helpers/rate_provider/rate_provider_v_100.vy 2847 2841 2273 1954 1974
curvefi/helpers/deposit_and_stake_zap/deposit_and_stake_zap_v_100.vy 2322 2316 1774 1611 1670
curvefi/governance/relayer/taiko/relayer_v_001.vy 2068 2064 1731 1510 1558
curvefi/governance/relayer/polygon_cdk/relayer_v_101.vy 1556 1523 1530 1324 1347
curvefi/governance/relayer/arb_orbit/relayer_v_101.vy 1266 1262 1242 1066 1115
curvefi/governance/relayer/op_stack/relayer_v_101.vy 1186 1182 1183 1014 1056
curvefi/governance/relayer/not_rollup/relayer_v_100.vy 1168 1153 1174 1011 1037
curvefi/governance/vault/vault_v_100.vy 964 941 862 823 839
curvefi/governance/relayer/relayer_v_100.vy 496 496 593 490 503
curvefi/governance/agent/agent_v_100.vy 541 541 430 402 406
curvefi/governance/agent/agent_v_101.vy 541 541 430 402 406

@harkal
harkal enabled auto-merge (squash) September 21, 2026 12:41
@harkal
harkal merged commit 8f6b7fb into vyperlang:master Sep 21, 2026
171 checks passed
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