[PATCH] Fix make check build warnings in libm

Siddhesh Poyarekar siddhesh@redhat.com
Mon Feb 18 03:41:00 GMT 2013


On Fri, Feb 15, 2013 at 09:44:34PM -0500, Carlos O'Donell wrote:
> 
> What about this commented out line?
> 
> Why are we using chk if all the assert's on chk are commented out?
> 
> >           chk |= (n ? mpn_sub_n : mpn_add_n)(d,ss,tmp,SZ);                    \
> >          /* assert(chk == 0); */                                              \
> 

I'm not sure why, but most mpn functions seem to return carry or
borrow if any or just zero.  It's been that way since inclusion
(1997), so it might make sense to just remove them.  I don't think any
of the inputs will cause an overflow and the checks were probably
there earlier for debugging.

Siddhesh



More information about the Libc-alpha mailing list