-
Notifications
You must be signed in to change notification settings - Fork 271
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 block_lookup
in Instruction
for block constant access
#77
Add block_lookup
in Instruction
for block constant access
#77
Conversation
Block
in Instruction
for block constant accessBlock
in Instruction
for block constant access
fb5c31c
to
8994742
Compare
Block
in Instruction
for block constant accessblock_context_lookup
in Instruction
for block constant access
block_context_lookup
in Instruction
for block constant accessblock_lookup
in Instruction
for block constant access
8f882fe
to
a35af4a
Compare
a35af4a
to
e2dbd08
Compare
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.
LGTM Thanks!
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.
LGTM
Could we merge this first? We have a |
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.
approve to unblock other tasks.
This PR aims to add the missing block constant functionality of
Instruction
.It adds a
block_table
to store all static block constant and up to previous 256 block hashes for block constant lookup. The table in real implementation could share the same instance columns withtx_table
once they are properly separate by selectors.This approach targets a straightforward implementation, in the future we could also adopt
EIP2935
mentioned in #75 to totally removeblock_table
.