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

implement EOF suite: EIP-3540, 3670, 4200, 4750, and 5450 #1413

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jangko
Copy link
Contributor

@jangko jangko commented Jan 10, 2023

I set the schedule to Cancun. Only need to modify 4 lines of code to move it back to Shanghai if needed.
It pass deployment test in test_eof.nim. I try to test it against Shanghai and EOF test vectors from ethereum/tests, but the test vectors itself have bugs.

Do not merge it until we have confirmation about which hard fork to include these EIPs and when it pass test from ethereum/tests and the test vectors have been fixed.

@jangko jangko force-pushed the eof-suite branch 3 times, most recently from c23f075 to e257f53 Compare January 15, 2023 07:38
@jangko
Copy link
Contributor Author

jangko commented Jan 16, 2023

test vectors from ethereum/tests have been fixed and we pass all tests cases of EIP-3540, 4670, 3860, 3855, 3651.

but test cases of EIP-4200(static jump), and EIP-4750(functions) still not available from ethereum/tests.

@jangko
Copy link
Contributor Author

jangko commented Mar 22, 2023

If EIP-1153(Transient Storage) accepted, then add TLOAD, and TSTORE to stack table.

TLOAD:
minStack: minStack(1, 1)
maxStack: maxStack(1, 1)

TSTORE
minStack: minStack(2, 0)
maxStack: maxStack(2, 0)

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.

1 participant