Hannes Reinecke wrote:
>
> Hi all,
>
> here is a patch for adding bn_div_words to asm/alpha.s. It got lost
> somehow, but without the assembler version won't compile.
>
> I'm not entirely familiar with the internal working of openssl, but to
> my untrained eye bn_asm.c:bn_div_words and bn_mulw.c:bn_div64 are
> looking suspiciously similar. Is there a special reason for it or can we
> just scrap one version ?
> (I suspect that's also why bn_div_words does not appear in asm/alpha.s;
> it just got renamed to bn_div64 hoping that all functions would call
> this and not bn_div_words. Evolutionary programming :-).
Looks to me like the whole of bn_mulw.c is redundant. bn_div_words and
bn_div64 are indeed identical, and bn_div64 is not actually used
anywhere, AFAICS.
So, I'm going to blow away bn_mulw.c, and rename bn_div64 to
bn_div_words in alpha.s.
Cheers,
Ben.
--
http://www.apache-ssl.org/ben.html
"My grandfather once told me that there are two kinds of people: those
who work and those who take the credit. He told me to try to be in the
first group; there was less competition there."
- Indira Gandhi
______________________________________________________________________
OpenSSL Project http://www.openssl.org
Development Mailing List [EMAIL PROTECTED]
Automated List Manager [EMAIL PROTECTED]
