On Mon, Dec 24, 2001 at 05:24:56PM -0800, Doug Kaufman wrote: > crypto/bn/bn_mul.c: This is a quick fix for DJGPP and CYGWIN, > but doesn't fix the underlying problem. BN_RECURSION appears to > be defined for all platforms in bn.h. bn_mul.c uses the function > bn_sub_part_words in a section of the code ifdef'd BN_RECURSION, but > bn_sub_part_words is compiled only when OPENSSL_NO_ASM is defined > or when __i386__ or __i386 are not defined. The function is NOT > defined in bn_asm.c. Can we remove the ifdef for all platforms for > bn_sub_part_words?
BN_RECURSION is the faster algorithm (though in practice it doesn't seem to matter that much, IIRC). The bn_sub_part_words situation obviously is a mess. The function is in bn_mul.c because we don't have asm implementations for non-Intel platforms, so the C version is also needed in some cases where bn_asm.c is not used ______________________________________________________________________ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED]
