core/vm: implement eof opcodes #30511
Open
Conversation
This file contains 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
MariusVanDerWijden
requested review from
fjl,
holiman,
karalabe,
gballet,
rjl493456442,
zsfelfoldi,
s1na and
lightclient
as code owners
|
Merged eof-validation, rebase time :) |
MariusVanDerWijden
force-pushed
the
eof-opcodes
branch
from
f6dd3c8 to
73a3a54
Compare
|
Rebased, looks like I broke some verkle stuff, will fix later |
holiman
reviewed
Oct 8, 2024
MariusVanDerWijden
force-pushed
the
eof-opcodes
branch
from
b864b15 to
e027042
Compare
In state tests when running prague system contracts the origin remains as the system account. Restore the prior tx context after running system contracts.
Restore state test transaction context
Add support for the Osaka fork and move all EOF activation and side effects to that fork.
shemnon
reviewed
Oct 16, 2024
When copying EOF contracts, make sure EXTCODECOPY only has access to the EOF magic, not just two bytes total.
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
based on #30418
rework of #29518
I could not test this properly yet, as the validity of the blockchain tests depend on the prague spec. In order to test this correctly, we need to upstream 6110 and 7002
Unfortunately I couldn't preserve the authorship, I think by using the coauthored tag, people will receive their due credit once this is merged.
Co-authored-by: lightclient [email protected]
Co-authored-by: shemnon [email protected]
Co-authored-by: holiman [email protected]