Skip to content

crypto: Drop the runtime-modulus Montgomery arithmetic - #1664

Open
chfast wants to merge 1 commit into
masterfrom
crypto/modarith-compile-time-mod
Open

crypto: Drop the runtime-modulus Montgomery arithmetic#1664
chfast wants to merge 1 commit into
masterfrom
crypto/modarith-compile-time-mod

crypto: Drop the runtime-modulus Montgomery arithmetic

7eb0a63
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Aug 19, 2026 in 1s

97.70% (-0.02%) compared to 9344010

View this Pull Request on Codecov

97.70% (-0.02%) compared to 9344010

Details

Codecov Report

❌ Patch coverage is 91.01124% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.70%. Comparing base (9344010) to head (7eb0a63).

Files with missing lines Patch % Lines
test/unittests/modarith_test.cpp 84.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1664      +/-   ##
==========================================
- Coverage   97.72%   97.70%   -0.02%     
==========================================
  Files         171      171              
  Lines       15636    15631       -5     
  Branches     3616     3616              
==========================================
- Hits        15280    15272       -8     
- Misses        269      272       +3     
  Partials       87       87              
Components Coverage Δ
core 96.05% <100.00%> (-0.07%) ⬇️
tooling 91.92% <ø> (ø)
tests 99.80% <85.45%> (-0.01%) ⬇️
Files with missing lines Coverage Δ
lib/evmone_precompiles/ecc.hpp 97.24% <100.00%> (ø)
lib/evmone_precompiles/modarith.hpp 96.25% <100.00%> (-3.75%) ⬇️
lib/evmone_precompiles/secp256r1.cpp 100.00% <100.00%> (ø)
test/unittests/secp256k1_test.cpp 100.00% <100.00%> (ø)
test/unittests/modarith_test.cpp 88.73% <84.00%> (-0.31%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.