tests and bench do not need linking with secp256k1.o about secp256k1 HOT 4 CLOSED

@sipa now that we have automake and can select the proper input files based on configure, would you be interested in moving the _impl.h -> _impl.c, then building the necessary objects based on make directives, rather than include magic?

That would help in this case, as well as in modularizing a bit. As an easy-to-see win, the test program could then run against all implementations at once.

If that's not a goal, then we can just take the shortcut here and link in the asm object in that case.

from secp256k1.

@theuni That breaks the compiler's ability to inline, which we're strongly relying upon. I haven't tested the performance hit, so I may be overestimating it.

from secp256k1.

@sipa Understood. I'd like to discuss that, but this issue isn't the place for it. I'll PR the simple fix for this one.

from secp256k1.

Fixed by #55.

from secp256k1.