search

briansmith / ring

Safe, fast, small crypto using Rust
Other
3.64k stars 682 forks source link

Add XTENSA architecture to target.h #2124

Closed juliankrieger closed 1 day ago

juliankrieger commented 3 days ago

In accordance to #2088

codecov[bot] commented 2 days ago

Codecov Report

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

Project coverage is 97.12%. Comparing base (fd06b00) to head (256df3c).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2124 +/- ## ========================================== - Coverage 97.12% 97.12% -0.01% ========================================== Files 151 151 Lines 20101 20101 Branches 447 447 ========================================== - Hits 19524 19523 -1 Misses 482 482 - Partials 95 96 +1 ```

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

briansmith commented 2 days ago

Thanks for doing this. This looks good to me but the commit message is not good. Please improve the commit message to something like "Fix build for Xtensa targets when using older versions of GCC."

juliankrieger commented 2 days ago

Ah, sorry. Consider it done.

juliankrieger commented 2 days ago

Done.