feat(spec-specs, tests): align EIP-8037 with forks/amsterdam GasCosts refactor#2737
Merged
spencer-tb merged 1 commit intoethereum:eips/amsterdam/eip-8037from Apr 21, 2026
Conversation
spencer-tb
force-pushed
the
eips/amsterdam/eip-8037
branch
from
de71eca to
aedeebd
Compare
spencer-tb
force-pushed
the
pr/eip-8037-amsterdam-compat
branch
from
fb5375b to
e956154
Compare
spencer-tb
changed the title
forks/amsterdam GasCosts refactor
spencer-tb
added
C-feat
spencer-tb
force-pushed
the
pr/eip-8037-amsterdam-compat
branch
from
e956154 to
5c359e3
Compare
spencer-tb
force-pushed
the
eips/amsterdam/eip-8037
branch
from
de71eca to
f2049fb
Compare
spencer-tb
force-pushed
the
pr/eip-8037-amsterdam-compat
branch
3 times, most recently
from
0f8464c to
76a0cdc
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## eips/amsterdam/eip-8037 #2737 +/- ##
==========================================================
Coverage ? 86.49%
==========================================================
Files ? 599
Lines ? 37765
Branches ? 3804
==========================================================
Hits ? 32666
Misses ? 4531
Partials ? 568
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Adapt EIP-8037 to forks/amsterdam's module-level-to-GasCosts refactor: rename GAS_X → GasCosts.X in 8037 files, with remappings: GAS_STORAGE_UPDATE → GasCosts.COLD_STORAGE_WRITE GAS_KECCAK256_PER_WORD → GasCosts.OPCODE_KECCACK256_PER_WORD GAS_SELF_DESTRUCT → GasCosts.OPCODE_SELFDESTRUCT_BASE GAS_CREATE → GasCosts.OPCODE_CREATE_BASE GAS_MEMORY → GasCosts.MEMORY_PER_WORD Tests: 977 pass.
spencer-tb
force-pushed
the
pr/eip-8037-amsterdam-compat
branch
from
76a0cdc to
f83131f
Compare
spencer-tb
merged commit c3dee65
into
ethereum:eips/amsterdam/eip-8037
23 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adapts the EIP-8037 branch to
forks/amsterdam's module level to GasCosts refactor (#2396). Files touched by EIP-8037 switch from the old module-levelGAS_Xconstants toGasCosts.Xattributes;GAS_STORAGE_UPDATEmaps toGasCosts.COLD_STORAGE_WRITE.