Skip to content

precompiles: Add EC mixed-add formula #1170

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

chfast
Copy link
Member

@chfast chfast commented Mar 14, 2025

Another -10% time reduction.

@chfast chfast added the precompiles Related to EVM precompiles label Mar 14, 2025
Copy link

codecov bot commented Mar 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.50%. Comparing base (26cdd85) to head (142b394).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1170      +/-   ##
==========================================
+ Coverage   94.48%   94.50%   +0.01%     
==========================================
  Files         169      169              
  Lines       18014    18057      +43     
==========================================
+ Hits        17021    17064      +43     
  Misses        993      993              
Flag Coverage Δ
eof_execution_spec_tests 20.84% <0.00%> (-0.05%) ⬇️
ethereum_tests 27.23% <95.74%> (+0.15%) ⬆️
ethereum_tests_silkpre 19.73% <95.65%> (+0.19%) ⬆️
execution_spec_tests 21.20% <0.00%> (-0.06%) ⬇️
unittests 91.68% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
lib/evmone_precompiles/bn254.cpp 100.00% <100.00%> (ø)
lib/evmone_precompiles/ecc.hpp 100.00% <100.00%> (ø)
lib/evmone_precompiles/secp256k1.cpp 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
precompiles Related to EVM precompiles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant