briansmith / ring

Safe, fast, small crypto using Rust
Other
3.77k stars 708 forks source link

poly1305 x86_64: Remove poly1305_vec.c. #2072

Closed briansmith closed 6 months ago

briansmith commented 6 months ago

On x86_64, when we have SSE 4.1, which is pretty much always the case, we use the integrated implementation. We also have the non-SSE implementation to fall back on. This middle one is infrequently used but creates difficulties with cross-compilation because of the use of the header file, which forces a dependency on stdlib.h we wouldn't otherwise need.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.79%. Comparing base (03c3593) to head (23b5553).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2072 +/- ## ========================================== - Coverage 96.79% 96.79% -0.01% ========================================== Files 145 144 -1 Lines 20743 20087 -656 Branches 244 228 -16 ========================================== - Hits 20078 19443 -635 + Misses 631 619 -12 + Partials 34 25 -9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.