Clean up dbl-64 rint, nearbyint.

diff --git a/ChangeLog b/ChangeLog
index 391bb6050f3df73c9e12b8ea32e45b1a2d5f93e1..b8f1fe53ca0d882d74f922168039a4c9b0875d49 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2012-03-13  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/ieee754/dbl-64/s_nearbyint.c (__nearbyint): Do not
+       manipulate bits before adding and subtracting TWO52[sx].
+       * sysdeps/ieee754/dbl-64/s_rint.c (__rint): Likewise.
+       * sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c (__nearbyint):
+       Likewise.
+       * sysdeps/ieee754/dbl-64/wordsize-64/s_rint.c (__rint): Likewise.
+
 2012-03-13  David S. Miller  <davem@davemloft.net>
 
        * sysdeps/sparc/Makefile: Remove rtld-global-offsets.sym handling.

diff --git a/sysdeps/ieee754/dbl-64/s_nearbyint.c b/sysdeps/ieee754/dbl-64/s_nearbyint.c
index d780150a907512ab727d1b13c4aa8d1eacd63def..60afafdeddbb2cd027ea7ad6e48b65400d48b316 100644 (file)
--- a/sysdeps/ieee754/dbl-64/s_nearbyint.c
+++ b/sysdeps/ieee754/dbl-64/s_nearbyint.c
@@ -38,18 +38,12 @@ double __nearbyint(double x)
 {
        fenv_t env;
        int32_t i0,j0,sx;
-       u_int32_t i,i1;
        double w,t;
-       EXTRACT_WORDS(i0,i1,x);
+       GET_HIGH_WORD(i0,x);
        sx = (i0>>31)&1;
        j0 = ((i0>>20)&0x7ff)-0x3ff;
-       if(j0<20) {
+       if(j0<52) {
            if(j0<0) {
-               if(((i0&0x7fffffff)|i1)==0) return x;
-               i1 |= (i0&0x0fffff);
-               i0 &= 0xfffe0000;
-               i0 |= ((i1|-i1)>>12)&0x80000;
-               SET_HIGH_WORD(x,i0);
                feholdexcept (&env);
                w = TWO52[sx]+x;
                t =  w-TWO52[sx];
@@ -57,32 +51,11 @@ double __nearbyint(double x)
                GET_HIGH_WORD(i0,t);
                SET_HIGH_WORD(t,(i0&0x7fffffff)|(sx<<31));
                return t;
-           } else {
-               i = (0x000fffff)>>j0;
-               if(((i0&i)|i1)==0) return x; /* x is integral */
-               i>>=1;
-               if(((i0&i)|i1)!=0) {
-                 if (j0==19)
-                   i1 = 0x40000000;
-                 else if (j0<18)
-                   i0 = (i0&(~i))|((0x20000)>>j0);
-                 else
-                   {
-                     i0 &= ~i;
-                     i1 = 0x80000000;
-                   }
-               }
            }
-       } else if (j0>51) {
+       } else {
            if(j0==0x400) return x+x;   /* inf or NaN */
            else return x;              /* x is integral */
-       } else {
-           i = ((u_int32_t)(0xffffffff))>>(j0-20);
-           if((i1&i)==0) return x;     /* x is integral */
-           i>>=1;
-           if((i1&i)!=0) i1 = (i1&(~i))|((0x40000000)>>(j0-20));
        }
-       INSERT_WORDS(x,i0,i1);
        feholdexcept (&env);
        w = TWO52[sx]+x;
        t = w-TWO52[sx];

diff --git a/sysdeps/ieee754/dbl-64/s_rint.c b/sysdeps/ieee754/dbl-64/s_rint.c
index c45d7848d7b0c570ee84b4ee822af417bea0a79d..84589091656208e95f776aa7e75e90ec62212765 100644 (file)
--- a/sysdeps/ieee754/dbl-64/s_rint.c
+++ b/sysdeps/ieee754/dbl-64/s_rint.c
@@ -33,49 +33,22 @@ double
 __rint(double x)
 {
        int32_t i0,j0,sx;
-       u_int32_t i,i1;
        double w,t;
-       EXTRACT_WORDS(i0,i1,x);
+       GET_HIGH_WORD(i0,x);
        sx = (i0>>31)&1;
        j0 = ((i0>>20)&0x7ff)-0x3ff;
-       if(j0<20) {
+       if(j0<52) {
            if(j0<0) {
-               if(((i0&0x7fffffff)|i1)==0) return x;
-               i1 |= (i0&0x0fffff);
-               i0 &= 0xfffe0000;
-               i0 |= ((i1|-i1)>>12)&0x80000;
-               SET_HIGH_WORD(x,i0);
                w = TWO52[sx]+x;
                t =  w-TWO52[sx];
                GET_HIGH_WORD(i0,t);
                SET_HIGH_WORD(t,(i0&0x7fffffff)|(sx<<31));
                return t;
-           } else {
-               i = (0x000fffff)>>j0;
-               if(((i0&i)|i1)==0) return x; /* x is integral */
-               i>>=1;
-               if(((i0&i)|i1)!=0) {
-                   if (j0==19)
-                     i1 = 0x40000000;
-                   else if (j0<18)
-                     i0 = (i0&(~i))|((0x20000)>>j0);
-                   else
-                     {
-                       i0 &= ~i;
-                       i1 = 0x80000000;
-                     }
-               }
            }
-       } else if (j0>51) {
+       } else {
            if(j0==0x400) return x+x;   /* inf or NaN */
            else return x;              /* x is integral */
-       } else {
-           i = ((u_int32_t)(0xffffffff))>>(j0-20);
-           if((i1&i)==0) return x;     /* x is integral */
-           i>>=1;
-           if((i1&i)!=0) i1 = (i1&(~i))|((0x40000000)>>(j0-20));
        }
-       INSERT_WORDS(x,i0,i1);
        w = TWO52[sx]+x;
        return w-TWO52[sx];
 }

diff --git a/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c b/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
index a3809984e2c9bc9dd87fa2b6979c1fa2221a082b..a58a6202efc383a0db9bc5a37cb148b96ae58ee3 100644 (file)
--- a/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
+++ b/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
@@ -41,29 +41,17 @@ __nearbyint(double x)
        j0 = ((i0>>52)&0x7ff)-0x3ff;
        if(__builtin_expect(j0<52, 1)) {
            if(j0<0) {
-             if((i0&UINT64_C(0x7fffffffffffffff))==0) return x;
-               uint64_t i = i0 & UINT64_C(0xfffffffffffff);
-               i0 &= UINT64_C(0xfffe000000000000);
-               i0 |= (((i|-i) >> 12) & UINT64_C(0x8000000000000));
-               INSERT_WORDS64(x,i0);
                libc_feholdexcept (&env);
                double w = TWO52[sx]+x;
                double t =  w-TWO52[sx];
                math_opt_barrier(t);
                libc_fesetenv (&env);
                return copysign(t, x);
-           } else {
-               uint64_t i = UINT64_C(0x000fffffffffffff)>>j0;
-               if((i0&i)==0) return x; /* x is integral */
-               i>>=1;
-               if((i0&i)!=0)
-                   i0 = (i0&(~i))|(UINT64_C(0x4000000000000)>>j0);
            }
        } else {
            if(j0==0x400) return x+x;   /* inf or NaN */
            else return x;              /* x is integral */
        }
-       INSERT_WORDS64(x,i0);
        libc_feholdexcept (&env);
        double w = TWO52[sx]+x;
        double t = w-TWO52[sx];

diff --git a/sysdeps/ieee754/dbl-64/wordsize-64/s_rint.c b/sysdeps/ieee754/dbl-64/wordsize-64/s_rint.c
index 20eef3c5ea40581c27ba8244dc21aa8dac46c53f..87b2339d438c97270c32a29939af84c1ee577682 100644 (file)
--- a/sysdeps/ieee754/dbl-64/wordsize-64/s_rint.c
+++ b/sysdeps/ieee754/dbl-64/wordsize-64/s_rint.c
@@ -38,28 +38,16 @@ __rint(double x)
        j0 = ((i0>>52)&0x7ff)-0x3ff;
        if(j0<52) {
            if(j0<0) {
-               if((i0 & UINT64_C(0x7fffffffffffffff))==0) return x;
-               uint64_t i = i0 & UINT64_C(0xfffffffffffff);
-               i0 &= UINT64_C(0xfffe000000000000);
-               i0 |= (((i|-i) >> 12) & UINT64_C(0x8000000000000));
-               INSERT_WORDS64(x,i0);
                double w = TWO52[sx]+x;
                double t =  w-TWO52[sx];
                EXTRACT_WORDS64(i0,t);
                INSERT_WORDS64(t,(i0&UINT64_C(0x7fffffffffffffff))|(sx<<63));
                return t;
-           } else {
-               uint64_t i = UINT64_C(0x000fffffffffffff)>>j0;
-               if((i0&i)==0) return x; /* x is integral */
-               i>>=1;
-               if((i0&i)!=0)
-                   i0 = (i0&(~i))|(UINT64_C(0x4000000000000)>>j0);
            }
        } else {
            if(j0==0x400) return x+x;   /* inf or NaN */
            else return x;              /* x is integral */
        }
-       INSERT_WORDS64(x,i0);
        double w = TWO52[sx]+x;
        return w-TWO52[sx];
 }