Replace 1L with (mp_limb_t) 1

X86-64 and x32 use sysdeps/i386/ldbl2mpn.c.  res_ptr is a pointer
to mp_limb_t, which is long for i386 and x86-64 and long long for
x32.  On x32, I got

../sysdeps/x86_64/../i386/ldbl2mpn.c: In function ‘__mpn_extract_long_double’:
../sysdeps/x86_64/../i386/ldbl2mpn.c:72:4: error: left shift count >= width of type [-Werror]
    res_ptr[N - 1] &= ~(1L << ((LDBL_MANT_DIG - 1) % BITS_PER_MP_LIMB));
    ^
cc1: all warnings being treated as errors

This patch replaces 1L with (mp_limb_t) 1.  Verified on x32, i686 and
x86-64 with GCC 4.8.3.

diff --git a/ChangeLog b/ChangeLog
index 799b21a..eda51f8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-19  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * sysdeps/i386/ldbl2mpn.c (__mpn_extract_long_double): Replace
+       1L with (mp_limb_t) 1.
+
 2014-12-17  Roland McGrath  <roland@hack.frob.com>
 
        * sysdeps/nptl/fork.c (__fork_generation_pointer): Variable moved ...

diff --git a/sysdeps/i386/ldbl2mpn.c b/sysdeps/i386/ldbl2mpn.c
index 09e3096..6f92833 100644 (file)
--- a/sysdeps/i386/ldbl2mpn.c
+++ b/sysdeps/i386/ldbl2mpn.c
@@ -69,7 +69,7 @@ __mpn_extract_long_double (mp_ptr res_ptr, mp_size_t size,
             for denormalized number.  If it is one, the number is according
             to Intel's specification an invalid number.  We make the
             representation unique by explicitly clearing this bit.  */
-         res_ptr[N - 1] &= ~(1L << ((LDBL_MANT_DIG - 1) % BITS_PER_MP_LIMB));
+         res_ptr[N - 1] &= ~((mp_limb_t) 1 << ((LDBL_MANT_DIG - 1) % BITS_PER_MP_LIMB));
 
          if (res_ptr[N - 1] != 0)
            {