merge from gcc
authorDJ Delorie <dj@redhat.com>
Tue, 13 Nov 2007 15:12:42 +0000 (15:12 +0000)
committerDJ Delorie <dj@redhat.com>
Tue, 13 Nov 2007 15:12:42 +0000 (15:12 +0000)
libiberty/ChangeLog
libiberty/floatformat.c

index 78e74e52e225961f35c9a16274de90b42e9975fe..1260eeba20c66d075132f40828e5f323d24b3dd5 100644 (file)
@@ -1,3 +1,10 @@
+2007-11-12  Joseph Myers  <joseph@codesourcery.com>
+
+       * floatformat.c (floatformat_ibm_long_double_is_valid): Fix
+       compiler warnings.
+       (floatformat_ibm_long_double): Use
+       floatformat_ibm_long_double_is_valid.
+
 2007-11-07  Joseph Myers  <joseph@codesourcery.com>
            Daniel Jacobowitz  <dan@codesourcery.com>
 
index 07af7264df499c932b3d07662513a4a86304cc14..cbf13ea3406076ec7efd052d408ac135ec2eda09 100644 (file)
@@ -283,7 +283,7 @@ floatformat_ibm_long_double_is_valid (const struct floatformat *fmt,
   bot_exp = get_field (ufrom + 8, hfmt->byteorder, hfmt->totalsize,
                       hfmt->exp_start, hfmt->exp_len);
 
-  if (top_exp == hfmt->exp_nan)
+  if ((unsigned long) top_exp == hfmt->exp_nan)
     top_nan = mant_bits_set (hfmt, ufrom);
 
   /* A NaN is valid with any low part.  */
@@ -292,11 +292,8 @@ floatformat_ibm_long_double_is_valid (const struct floatformat *fmt,
 
   /* An infinity, zero or denormal requires low part 0 (positive or
      negative).  */
-  if (top_exp == hfmt->exp_nan || top_exp == 0)
+  if ((unsigned long) top_exp == hfmt->exp_nan || top_exp == 0)
     {
-      unsigned int mant_bits, mant_off;
-      int mant_bits_left;
-
       if (bot_exp != 0)
        return 0;
 
@@ -318,7 +315,7 @@ floatformat_ibm_long_double_is_valid (const struct floatformat *fmt,
       /* The bottom part is 0 or denormal.  Determine which, and if
         denormal the first two set bits.  */
       int first_bit = -1, second_bit = -1, cur_bit;
-      for (cur_bit = 0; cur_bit < hfmt->man_len; cur_bit++)
+      for (cur_bit = 0; (unsigned int) cur_bit < hfmt->man_len; cur_bit++)
        if (get_field (ufrom + 8, hfmt->byteorder, hfmt->totalsize,
                       hfmt->man_start + cur_bit, 1))
          {
@@ -363,7 +360,7 @@ const struct floatformat floatformat_ibm_long_double =
   floatformat_big, 128, 0, 1, 11, 1023, 2047, 12, 52,
   floatformat_intbit_no,
   "floatformat_ibm_long_double",
-  floatformat_always_valid,
+  floatformat_ibm_long_double_is_valid,
   &floatformat_ieee_double_big
 };
 \f