Skip to content

chore: use revm glam-devnet-6 (b55020b2)#402

Draft
rakita wants to merge 1 commit into
mainfrom
glam-devnet-6
Draft

chore: use revm glam-devnet-6 (b55020b2)#402
rakita wants to merge 1 commit into
mainfrom
glam-devnet-6

Conversation

@rakita

@rakita rakita commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Pins the revm dependencies to the glam-devnet-6 branch of bluealloy/revm at commit b55020b237400aa42ce48fcaefdf5cdc255b5558, and fixes the breakages introduced by that bump.

Changes

  • Cargo.toml: switch all revm-* workspace deps from crates.io 41.0.0 to git deps pinned at rev b55020b2.
  • crates/revmc-context/src/lib.rs: EvmContext field offsets shifted by +8 because Gas grew by one u64. Updated the layout static asserts (spec_id 113→121, resume_at 120→128, calldatasize 160→168). The JIT codegen reads these via offset_of!, so it adjusts automatically; only the guard asserts needed updating.
  • crates/revmc-codegen/src/compiler/translate/mod.rs: revm's GasTracker gained a state_gas_spilled field (EIP-8037). Mirrored it in the pf::Gas offset_of! helper struct so the size_of assertion and tracker.remaining offset stay in sync.
  • crates/revmc-statetest/src/compiled.rs: CfgEnv::is_eip7708_delayed_burn_disabled was renamed to is_eip8246_delayed_clear_disabled.

Verification

  • cargo check --workspace
  • cargo nextest run -p revmc-codegen -p revmc-context -p revmc-builtins — 2189 passed, 0 failed ✅
  • CLI smoke test (cargo r -- run usdc_proxy) runs clean ✅

Note

This pins revm to a devnet branch via git rev and is intended for the glam-devnet-6 track, not for merging into main.

Point revm-* deps at the bluealloy/revm glam-devnet-6 branch
(rev b55020b2) and fix the breakages from that bump:

- EvmContext field offsets shifted +8 (Gas grew by one u64); update the
  layout static asserts (spec_id 121, resume_at 128, calldatasize 168).
- Mirror revm's GasTracker gaining the state_gas_spilled field in the
  codegen offset_of! helper struct so size/layout stays in sync.
- Cfg method is_eip7708_delayed_burn_disabled was renamed to
  is_eip8246_delayed_clear_disabled in the statetest runner.
@rakita
rakita requested a review from DaniPopes as a code owner June 29, 2026 16:42
@codspeed-hq

codspeed-hq Bot commented Jun 29, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 71 untouched benchmarks


Comparing glam-devnet-6 (d783584) with main (b5b60c1)

Open in CodSpeed

@DaniPopes
DaniPopes marked this pull request as draft June 29, 2026 21:08
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