Skip to content

Conversation

@rkrasiuk
Copy link
Collaborator

Description

Update EIP-7825 constant. Ref ethereum/EIPs#9986

@codspeed-hq
Copy link

codspeed-hq bot commented Jul 22, 2025

CodSpeed Performance Report

Merging #2753 will not alter performance

Comparing rkrasiuk/update-7825-constant (31ff00e) with main (bd14463)

Summary

✅ 171 untouched benchmarks

@rakita
Copy link
Member

rakita commented Jul 22, 2025

For failed tests, remove files from op-revm/tests/testdata and rum test again. Thos would regenerate fixtures that were changed

@rakita rakita merged commit bedcba7 into main Jul 22, 2025
28 checks passed
rezbera added a commit to berachain/bera-reth that referenced this pull request Jul 28, 2025
We used a variable in revm for the PoL Tx gas limit. This was updated
and we inherited the wrong gas limit.
bluealloy/revm#2753

Further e2e tests in subsequent prs will cover this regression vector.

The actual gas limit used in the revm system tx is still 30million
https://github.com/bluealloy/revm/blob/6a736b39e039200dbb6f254aa641875235b37e32/crates/handler/src/system_call.rs#L65
@rakita rakita deleted the rkrasiuk/update-7825-constant branch August 7, 2025 12:24
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