lshift.asm bug

Torbjorn Granlund tg-this-will-bounce-but-I-am-subscribed-to-the-list-honest at swox.com
Wed Mar 7 15:08:47 CET 2007


"delta trinity" <deltatrinity at hotmail.com> writes:

  I was looking at the asm source code for the k7 (gmp 4.2.1) mmx
  lshift.asm, and found a bug
  
[snip]

  It is to note that the default (shipped with GMP 4.2.1) is
  UNROLL_COUNT=32, so this bug is not seen under default conditions.
  
If you're talking about mpn/x86/k7/mmx, then I suppose the default
count is 16, also safe.

Instead of fixing the typo, I think I'll make the code handle up to
32x unrolling, for simplicity, which is already way too much.  (And
perhaps lower the default to 8.)

-- 
Torbjörn


More information about the gmp-bugs mailing list