-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat(levm): add precompiled contracts addresses to cache #1473
feat(levm): add precompiled contracts addresses to cache #1473
Conversation
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
for i in 1..10 { | ||
default_touched_accounts.insert(Address::from_low_u64_be(i)); | ||
} | ||
|
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.
Can't we iterate the constant we have in precompiles.rs
with the list of precompile addresses?
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.
Added this as a to-do comment in Add todo coment
@@ -138,6 +138,7 @@ impl VM { | |||
|
|||
let mut default_touched_storage_slots: HashMap<Address, HashSet<H256>> = HashMap::new(); | |||
|
|||
// Add access lists contents to cache |
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.
Is this meant for this PR?
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.
It's not strictly related, but makes more understandable the code IMO
Motivation
Precompiles are not implemented yet, but some tests test sideways those addresses are cached, so this makes those tests (8 tests) pass.
Description
Just adds addresses between 1 and 9 to the cached addresses.