Make the num module optional #119

pull sipa wants to merge 2 commits into bitcoin-core:master from sipa:nonenum changing 12 files +139 −101
  1. sipa commented at 0:31 am on November 28, 2014: contributor

    Build on top of #117, and makes num optional entirely.

    Having num (with one implementation: GMP) available means faster field and scalar inverses, and is required for the endomorphism optimization.

    Benchmarks (with –with-field=64bit_asm) for verification:

    • –with-num=gmp –enable-endomorphism: 253k cycles
    • –with-num=gmp –disable-endomorphism: 345k cycles
    • –with-num=none –disable-endomorphism: 432k cycles

    (secp256k1 in OpenSSL needs 1523k cycles, Ed25519 needs 231k cycles)

  2. sipa cross-referenced this on Nov 28, 2014 from issue Avoid field inverse for r == x comparison by sipa
  3. sipa force-pushed on Nov 30, 2014
  4. sipa force-pushed on Nov 30, 2014
  5. sipa force-pushed on Nov 30, 2014
  6. sipa commented at 10:51 pm on November 30, 2014: contributor
    @theuni Care to review the build changes here (last commit only)?
  7. sipa force-pushed on Dec 1, 2014
  8. Make constant initializers independent from num 659b554d7b
  9. Make num optional 597128d389
  10. sipa force-pushed on Dec 1, 2014
  11. gmaxwell commented at 2:28 pm on December 2, 2014: contributor
    ACK
  12. sipa merged this on Dec 2, 2014
  13. sipa closed this on Dec 2, 2014

  14. sipa referenced this in commit bd313f7d6e on Dec 2, 2014
  15. sipa cross-referenced this on Dec 2, 2014 from issue unable to configure on Ubuntu 14.04 by gdassori


sipa gmaxwell


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-30 09:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me