Skip to content

deps: bump tempo to TIP-1016 (rev 2e6e9d1)#14359

Closed
decofe wants to merge 8 commits intomasterfrom
tempo-tip1016
Closed

deps: bump tempo to TIP-1016 (rev 2e6e9d1)#14359
decofe wants to merge 8 commits intomasterfrom
tempo-tip1016

Conversation

@decofe
Copy link
Copy Markdown
Contributor

@decofe decofe commented Apr 17, 2026

Upstream PR: tempoxyz/tempo#2684

Bumps tempo crates to rev 2e6e9d1 which patches revm for TIP-1016 gas accounting changes (InitialAndFloorGas handler parameter).

Patches:

  • revmbluealloy/revm@a1a1824
  • op-revmfoundry-rs/op-revm@66388e6
  • reth-coreparadigmxyz/reth-core@6b12498 (Bytecode::new_analyzed unsafe)

Adds gas_limit() and state_gas_used() to AnvilStorageProvider for the updated PrecompileStorageProvider trait.

Prompted by: rusowsky

Patches revm to bluealloy/revm@a1a1824 and op-revm to
foundry-rs/op-revm@66388e6 for the InitialAndFloorGas handler change.
Patches reth-core to paradigmxyz/reth-core@6b12498 for the
Bytecode::new_analyzed unsafe change.

Adds gas_limit() and state_gas_used() to AnvilStorageProvider to
satisfy the updated PrecompileStorageProvider trait.

Co-authored-by: 0xrusowsky <90208954+0xrusowsky@users.noreply.github.com>
Co-authored-by: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d97c8-3033-770a-bd6c-714c2a8393d5
decofe and others added 3 commits April 17, 2026 05:33
- Replace deprecated OpCode::new_unchecked with new_or_unknown
- Add gas_refunded field to PrecompileOutput struct literals
- Allow foundry-rs/op-revm and paradigmxyz/reth-core git sources in deny.toml

Co-Authored-By: 0xrusowsky <90208954+0xrusowsky@users.noreply.github.com>
Co-Authored-By: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d97c8-3033-770a-bd6c-714c2a8393d5
Updates patch revisions to match tempo/tip1016 HEAD:
- revm: a1a1824 → ea8d1f5
- op-revm: 66388e6 → 780b812
- tempo: 2e6e9d1 → 4092dfe

Fixes CreateInputs::new reservoir parameter for the new revm.

Co-Authored-By: 0xrusowsky <90208954+0xrusowsky@users.noreply.github.com>
Co-Authored-By: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d97c8-3033-770a-bd6c-714c2a8393d5
Tempo's [patch.crates-io] now includes all revm sub-crates, preventing
the invariant workflow's patch propagation from clobbering foundry's
patches.

Co-Authored-By: 0xrusowsky <90208954+0xrusowsky@users.noreply.github.com>
Co-Authored-By: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d97c8-3033-770a-bd6c-714c2a8393d5
mablr
mablr previously approved these changes Apr 17, 2026
zerosnacks
zerosnacks previously approved these changes Apr 17, 2026
@decofe decofe dismissed stale reviews from zerosnacks and mablr via 071ada3 April 17, 2026 12:14
Copy link
Copy Markdown
Collaborator

@mablr mablr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems messed-up.

It reverted #14365.

@zerosnacks zerosnacks marked this pull request as draft April 17, 2026 13:56
@zerosnacks zerosnacks added the T-blocked Type: blocked label Apr 17, 2026
@zerosnacks
Copy link
Copy Markdown
Member

Temporarily marking as blocked whilst this is in draft to prevent eager merging

@zerosnacks
Copy link
Copy Markdown
Member

zerosnacks commented Apr 17, 2026

It seems messed-up.

It reverted #14365.

Ah, thats not what I wanted; fixed manually, bumped to later commit in the PR: tempoxyz/tempo#2684

Fixed 😄

@mablr mablr marked this pull request as ready for review April 17, 2026 14:06
mablr
mablr previously approved these changes Apr 17, 2026
Copy link
Copy Markdown
Collaborator

@mablr mablr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be good now

- revm 37.0.0 → 38.0.0 (crates.io release, no more git patches)
- revm-inspectors 0.38.1 → 0.39.0
- alloy-evm 0.32.0 → 0.33.1
- tempo crates rev f873f0e → c095527 (tip1016 head)
- foundry-fork-db, op-revm, foundry-rs/optimism bumped via branches
- fix OpCode::new_or_unknown → new_unchecked (API change in revm 38)
- add missing Evm::cfg_env impl (new trait method in alloy-evm 0.33.1)

Co-Authored-By: 0xrusowsky <90208954+0xrusowsky@users.noreply.github.com>
mablr
mablr previously approved these changes Apr 17, 2026
@0xrusowsky
Copy link
Copy Markdown
Contributor

done in #14436

@0xrusowsky 0xrusowsky closed this Apr 24, 2026
@github-project-automation github-project-automation Bot moved this to Done in Foundry Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-blocked Type: blocked

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants