Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EIP4758 test cases #1024

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

EIP4758 test cases #1024

wants to merge 2 commits into from

Conversation

jwasinger
Copy link
Contributor

@jwasinger jwasinger commented Apr 8, 2022

  • SENDALL with currently-executing contract as the beneficiary returns execution to the calling frame and leaves the contract which executed SENDALL in the state after the transaction finishes.
  • SENDALL with non-self beneficiary transfers the balance of the invoking account to the beneficiary, ends execution of the current frame. The account which executed SENDALL remains in the state after the transaction finishes.

@winsvega
Copy link
Collaborator

you can use lllc or yul in code src
this will also require to updated all our existing tests for selfdestruct when eip is accepted

Copy link
Collaborator

@winsvega winsvega left a comment

Choose a reason for hiding this comment

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

need to move test to EIPTests subfolder.
need to upgrade network naming to Merge+4758
index index section can be removed as it is -1 by default
src code can be used in yul or lllc

"gas" : -1,
"value" : -1
},
"network" : [">=EIP4758"],
Copy link
Collaborator

Choose a reason for hiding this comment

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

need to convert to Merge+4758

},
"expect" : [
{
"indexes" : {
Copy link
Collaborator

Choose a reason for hiding this comment

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

indexes can be removed as it is -1 by default

"095e7baea6a6c7c4c2dfeb977efac326af552d87" : {
"balance" : "1000000000000000000",
"// code" : "PUSH20 095e7baea6a6c7c4c2dfeb977efac326af552d87 SELFDESTRUCT",
"code" : ":raw 0x73095e7baea6a6c7c4c2dfeb977efac326af552d87ff",
Copy link
Collaborator

Choose a reason for hiding this comment

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

can use lllc here or yul

@gumb0
Copy link
Member

gumb0 commented Jan 20, 2023

Would be good to add tests that check whether execution stops after SENDALL

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.

3 participants