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

sipa commented at 12: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)
sipa force-pushed on Nov 30, 2014
sipa force-pushed on Nov 30, 2014
sipa force-pushed on Nov 30, 2014
sipa commented at 10:51 PM on November 30, 2014: contributor

@theuni Care to review the build changes here (last commit only)?
sipa force-pushed on Dec 1, 2014
Make constant initializers independent from num 659b554d7b
Make num optional 597128d389
sipa force-pushed on Dec 1, 2014
gmaxwell commented at 2:28 PM on December 2, 2014: contributor

ACK
sipa merged this on Dec 2, 2014
sipa closed this on Dec 2, 2014
sipa referenced this in commit bd313f7d6e on Dec 2, 2014

Contributors