-
Notifications
You must be signed in to change notification settings - Fork 726
Take aws-lc-rs 1.10 #2147
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
Take aws-lc-rs 1.10 #2147
Conversation
Benchmark resultsInstruction countsSignificant differencesThere are no significant instruction count differences Other differencesClick to expand
Wall-timeSignificant differencesClick to expand
Other differencesClick to expand
Additional informationCheckout details:
|
Sweet. This is the faster RSA impl using AVX512. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2147 +/- ##
=======================================
Coverage 94.66% 94.66%
=======================================
Files 102 102
Lines 23434 23435 +1
=======================================
+ Hits 22183 22184 +1
Misses 1251 1251 ☔ View full report in Codecov by Sentry. |
Cargo.lock
Outdated
@@ -1435,7 +1435,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" | |||
checksum = "4979f22fdb869068da03c9f7528f8297c6fd2606bc3a4affe42e6a823fdb8da4" | |||
dependencies = [ | |||
"cfg-if", | |||
"windows-targets 0.52.6", | |||
"windows-targets 0.48.5", |
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.
I wonder what caused this...
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.
Yes, very odd. Seems deterministic for cargo update -p aws-lc-rs
. If I do an unconstrained cargo update
it doesn't happen. It seems to be a result of accidentally keeping back aws-lc-fips-sys -- folded into this commit.
This will shortly be removed.
5caaeb0
to
31f0c59
Compare
No description provided.