Skip to content

core, core/vm: implement EIP-7708#33645

Merged
rjl493456442 merged 7 commits into
ethereum:masterfrom
zsfelfoldi:eip-7708
Mar 23, 2026
Merged

core, core/vm: implement EIP-7708#33645
rjl493456442 merged 7 commits into
ethereum:masterfrom
zsfelfoldi:eip-7708

Conversation

@zsfelfoldi
Copy link
Copy Markdown
Contributor

@zsfelfoldi zsfelfoldi commented Jan 19, 2026

This PR implements EIP-7708 according to the latest "rough consensus":

ethereum/EIPs#9003
https://github.com/etan-status/EIPs/blob/fl-ethlogs/EIPS/eip-7708.md

@zsfelfoldi zsfelfoldi changed the title core, core/vm: implement EIP-7708 (WIP) core, core/vm: implement EIP-7708 Jan 20, 2026
Comment thread core/types/log.go Outdated
Comment thread core/types/log.go Outdated
@jwasinger
Copy link
Copy Markdown
Contributor

Some changes just got merged here which include changes to how self destruct logs are emitted.

@zsfelfoldi
Copy link
Copy Markdown
Contributor Author

I updated according to the latest specs.

Comment thread core/vm/instructions.go Outdated
Comment thread core/vm/instructions.go Outdated
@jwasinger
Copy link
Copy Markdown
Contributor

We probably will adopt this too once it gets approved: ethereum/EIPs#11126 .

@fjl fjl added the amsterdam label Jan 22, 2026
@fjl fjl mentioned this pull request Jan 22, 2026
7 tasks
yperbasis added a commit to erigontech/erigon that referenced this pull request Jan 27, 2026
See https://eips.ethereum.org/EIPS/eip-7708. Based on
ethereum/go-ethereum#33645.

---------

Co-authored-by: Zsolt Felfoldi <zsfelfoldi@gmail.com>
@jwasinger
Copy link
Copy Markdown
Contributor

we can apply #33851 here.

@jwasinger
Copy link
Copy Markdown
Contributor

jwasinger commented Mar 2, 2026

we can apply #33851 here.

can apply this change to the PR once we prepare a devnet 3 branch. It's not part of devnet 2.

@jwasinger jwasinger mentioned this pull request Mar 2, 2026
@jwasinger jwasinger force-pushed the eip-7708 branch 3 times, most recently from 2c8f5db to 5b200bb Compare March 3, 2026 21:08
Comment thread core/state/statedb.go
@rjl493456442
Copy link
Copy Markdown
Member

One nitpick, otherwise lgtm

Copy link
Copy Markdown
Member

@MariusVanDerWijden MariusVanDerWijden left a comment

Choose a reason for hiding this comment

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

SGTM

zsfelfoldi and others added 3 commits March 23, 2026 21:28
Co-authored-by: Jared Wasinger <j-wasinger@hotmail.com>
Co-authored-by: raxhvl <raxhvl@users.noreply.github.com>
Copy link
Copy Markdown
Member

@MariusVanDerWijden MariusVanDerWijden left a comment

Choose a reason for hiding this comment

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

lgtm

@rjl493456442 rjl493456442 merged commit b87340a into ethereum:master Mar 23, 2026
4 of 6 checks passed
@rjl493456442 rjl493456442 added this to the 1.17.2 milestone Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants