We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
since the window is lower than u64::MAX we can use the lowest word and use u64 modulo operation instead of U256 for better performance.
The text was updated successfully, but these errors were encountered:
Hey! Can I take this?
Sorry, something went wrong.
Sure! The relevant operation is in https://github.com/bluealloy/revm/blob/main/crates/revm/src/context.rs#L134
Is this issue still relevant? as the operation has been removed in #1450 (sorry if the question sounds dumb)
Still is relevant, change Host and Database trait to use u64 and whatever breaks after it
Sorry for the delay,I somehow broke a lot of things ..still trying to fix those..will try to make the PR soon
Successfully merging a pull request may close this issue.
since the window is lower than u64::MAX we can use the lowest word and use u64 modulo operation instead of U256 for better performance.
The text was updated successfully, but these errors were encountered: