-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Add EIP: SLOT precompile #9141
base: master
Are you sure you want to change the base?
Add EIP: SLOT precompile #9141
Conversation
File
|
Is there a reason why it needs to be precompile and not just a new op code? |
I have a bit on this in the rationale, it prevents the EVM from growing and allows L2s to decide whether they need to implement it. See also this comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All Reviewers Have Approved; Performing Automatic Merge...
The commit 5cbf03c (as a parent of 77a8538) contains errors. |
seems like you have reverted everything with your latest commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All Reviewers Have Approved; Performing Automatic Merge...
Head branch was pushed to by a user without write access
This EIP proposes to add a new precompile that returns the corresponding slot for the current block.