math: fix remquo.c when x==-y and a subnormal remainder bug as well
authornsz <nsz@port70.net>
Mon, 7 May 2012 22:22:56 +0000 (00:22 +0200)
committernsz <nsz@port70.net>
Mon, 7 May 2012 22:22:56 +0000 (00:22 +0200)
backported fix from freebsd:
http://svnweb.FreeBSD.org/base?view=revision&revision=233973

src/math/remquo.c
src/math/remquof.c
src/math/remquol.c

index e92984e..085466e 100644 (file)
@@ -44,7 +44,7 @@ double remquo(double x, double y, int *quo)
                        goto fixup;
                }
                if (lx == ly) {            /* |x| = |y| return x*0 */
-                       *quo = 1;
+                       *quo = sxy ? -1 : 1;
                        return Zero[(uint32_t)sx>>31];
                }
        }
@@ -127,6 +127,7 @@ double remquo(double x, double y, int *quo)
 
        /* convert back to floating value and restore the sign */
        if ((hx|lx) == 0) {  /* return sign(x)*0 */
+               q &= 0x7fffffff;
                *quo = sxy ? -q : q;
                return Zero[(uint32_t)sx>>31];
        }
@@ -144,10 +145,10 @@ double remquo(double x, double y, int *quo)
                        hx >>= n;
                } else if (n <= 31) {
                        lx = (hx<<(32-n))|(lx>>n);
-                       hx = sx;
+                       hx = 0;
                } else {
                        lx = hx>>(n-32);
-                       hx = sx;
+                       hx = 0;
                }
        }
 fixup:
index 11569ce..536a050 100644 (file)
@@ -41,7 +41,7 @@ float remquof(float x, float y, int *quo)
                q = 0;
                goto fixup;
        } else if(hx==hy) {  /* |x| = |y| return x*0*/
-               *quo = 1;
+               *quo = sxy ? -1 : 1;
                return Zero[(uint32_t)sx>>31];
        }
 
@@ -92,6 +92,7 @@ float remquof(float x, float y, int *quo)
 
        /* convert back to floating value and restore the sign */
        if (hx == 0) {                             /* return sign(x)*0 */
+               q &= 0x7fffffff;
                *quo = sxy ? -q : q;
                return Zero[(uint32_t)sx>>31];
        }
index 721231b..a2e1172 100644 (file)
@@ -94,7 +94,7 @@ long double remquol(long double x, long double y, int *quo)
                        goto fixup;       /* |x|<|y| return x or x-y */
                }
                if (ux.bits.manh == uy.bits.manh && ux.bits.manl == uy.bits.manl) {
-                       *quo = 1;
+                       *quo = sxy ? -1 : 1;
                        return Zero[sx];  /* |x|=|y| return x*0*/
                }
        }
@@ -152,6 +152,7 @@ long double remquol(long double x, long double y, int *quo)
 
        /* convert back to floating value and restore the sign */
        if ((hx|lx) == 0) {  /* return sign(x)*0 */
+               q &= 0x7fffffff;
                *quo = sxy ? -q : q;
                return Zero[sx];
        }