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
evm: add sha512-256 precompile #1408
Conversation
91f529c
to
e312751
Compare
Codecov Report
@@ Coverage Diff @@
## main #1408 +/- ##
==========================================
+ Coverage 56.42% 56.52% +0.10%
==========================================
Files 128 130 +2
Lines 9198 9238 +40
==========================================
+ Hits 5190 5222 +32
- Misses 3966 3974 +8
Partials 42 42
|
029a936
to
8fefeb6
Compare
69a389a
to
145bb3e
Compare
use super::{record_linear_cost, PrecompileResult}; | ||
|
||
pub(super) fn call_sha512_256(handle: &mut impl PrecompileHandle) -> PrecompileResult { | ||
// Costs were computed by benchmarking and comparing to SHA256 and using the already set SHA256 costs (unsure how those were obtained). |
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.
(unsure how those were obtained)
I believe those come from Ethereum.
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.
Ah thanks, indeed. Updated the comment.
145bb3e
to
733096f
Compare
No description provided.