-
Notifications
You must be signed in to change notification settings - Fork 17
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
sha3 benchmark #324
sha3 benchmark #324
Conversation
Pull Request Test Coverage Report for Build 3881572104
💛 - Coveralls |
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.
Thanks! Looks good so far. Could we also add SHAKE (https://tech.cryspen.com/hacl-packages/c/main/hacl/hash/sha3.html#shake) to complete all of HACL SHA-3? (I would suggest to tackle all EverCrypt hashes separately.)
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.
We should also add comparison with OpenSSL. But we can do this in a follow up.
Nore that the comparison of benchmarks with main doesn't work because the sha3 benchmark doesn't exist there (same in #323). @pnmadelaine can you change the comparison script to run all benchmarks that are on main, not all that are on the branch?
benchmark comparison is fixed in #325 |
99efb92
to
1b5cf42
Compare
@duesee benchmarks for SHAKE are now implemented |
1b5cf42
to
1635ca6
Compare
1635ca6
to
b870774
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.
Thanks for making the changes! Looks good to me 👍
Add a benchmark for Sha3