Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make scalar/field choice depend on C-detected __int128 availability #793

Merged
merged 2 commits into from
Aug 12, 2020
Merged

Make scalar/field choice depend on C-detected __int128 availability #793

merged 2 commits into from
Aug 12, 2020

Commits on Aug 10, 2020

  1. Add SECP256K1_FE_STORAGE_CONST_GET to 5x52 field 

    So far this has not been needed, as it's only used by the static precomputation
which always builds with 32-bit fields.

This prepares for the ability to have __int128 detected on the C side, breaking
that restriction.
    @sipa
    sipa committed Aug 10, 2020
    Configuration menu
    Copy the full SHA
    0d7727f View commit details
    Browse the repository at this point in the history

  2. Autodetect __int128 availability on the C side 

    Instead of supporting configuration of the field and scalar size independently,
both are now controlled by the availability of a 64x64->128 bit multiplication
(currently only through __int128). This is autodetected from the C code through
__SIZEOF_INT128__, but can be overridden using configure's
--with-test-override-wide-multiply, or by defining
USE_FORCE_WIDEMUL_{INT64,INT128} manually.
    @sipa
    sipa committed Aug 10, 2020
    Configuration menu
    Copy the full SHA
    79f1f7a View commit details
    Browse the repository at this point in the history