Conversation
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
cpu
left a comment
There was a problem hiding this comment.
FWIW I got this branch building locally to fiddle with review feedback using this small patch: 7ae2f2b
What kind of test coverage did you have in-mind for this to be ready to go? There's some low-level ML-DSA test vectors recently added to Wycheproof (C2SP/wycheproof#146) but I'm not sure how well they'll map into this context.
|
I was thinking we could do a handshake with a freshly generated key? I guess that would require rcgen support maybe. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2579 +/- ##
=======================================
Coverage 94.81% 94.81%
=======================================
Files 96 96
Lines 21826 21826
Branches 614 614
=======================================
Hits 20695 20695
Misses 1003 1003
Partials 128 128 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
742dadc to
1b1ce27
Compare
966b6fd to
40bb6a4
Compare
|
Fixes #2577.
Proposed release notes
aws-lc-rs-unstablefeature is enabled.