i386.c (ix86_split_fp_branch): Remove unused variables.

	* config/i386/i386.c (ix86_split_fp_branch): Remove unused
	variables.
	(ix86_fp_compare_mode): Add unused attribute.
	(ix86_expand_fp_movcc): Remove unused variable.
	(ix86_expand_builtin): Make fcode unsigned.

	* expr.c (MOVE_BY_PIECES_P): Cast MOVE_RATIO to unsigned.
	(expand_expr): Add cast to avoid signed warning.
	(store_field): Likewise.
	(store_constructor_field): Likewise.
	(store_constructor): Likewise.
	(store_expr): Likewise.
	(clear_storage): Likewise.
	(emit_group_store): Likewise.
	(emit_group_load): Likewise.

From-SVN: r39000

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d8a1822de5d459a3fc29a9a32fa8495bb8831dce..8ec52e193e871b1b4cbbef28bd01c25a1465ee00 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,21 @@
+2001-01-14  Andreas Jaeger  <aj@suse.de>
+
+       * config/i386/i386.c (ix86_split_fp_branch): Remove unused
+       variables.
+       (ix86_fp_compare_mode): Add unused attribute.
+       (ix86_expand_fp_movcc): Remove unused variable.
+       (ix86_expand_builtin): Make fcode unsigned.
+
+       * expr.c (MOVE_BY_PIECES_P): Cast MOVE_RATIO to unsigned.
+       (expand_expr): Add cast to avoid signed warning.
+       (store_field): Likewise.
+       (store_constructor_field): Likewise.
+       (store_constructor): Likewise.
+       (store_expr): Likewise.
+       (clear_storage): Likewise.
+       (emit_group_store): Likewise.
+       (emit_group_load): Likewise.
+
 2001-01-13  Joseph S. Myers  <jsm28@cam.ac.uk>
 
        * c-common.c: Move format checking code to ...

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index da26760d9be021803d89b20114a40af2d0642cb1..eb8a852272c337abd058c558471f79904bdf1e76 100644 (file)
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -1,5 +1,5 @@
 /* Subroutines used for code generation on IA-32.
-   Copyright (C) 1988, 1992, 1994, 1995, 1996, 1997, 1998, 1999, 2000
+   Copyright (C) 1988, 1992, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001
    Free Software Foundation, Inc.
 
 This file is part of GNU CC.
@@ -4651,7 +4651,7 @@ ix86_expand_int_compare (code, op0, op1)
 
 enum machine_mode
 ix86_fp_compare_mode (code)
-     enum rtx_code code;
+     enum rtx_code code ATTRIBUTE_UNUSED;
 {
   /* ??? In order to make all comparisons reversible, we do all comparisons
      non-trapping when compiling for IEEE.  Once gcc is able to distinguish
@@ -5355,7 +5355,6 @@ ix86_split_fp_branch (condition, op1, op2, target1, target2, tmp)
   rtx second, bypass;
   rtx label = NULL_RTX;
   enum rtx_code code = GET_CODE (condition);
-  enum rtx_code bypass_code, second_code, first;
 
   if (target2 != pc_rtx)
     {
@@ -5811,7 +5810,6 @@ ix86_expand_fp_movcc (operands)
      rtx operands[];
 {
   enum rtx_code code;
-  enum machine_mode mode;
   rtx tmp;
   rtx compare_op;
 
@@ -8156,7 +8154,7 @@ ix86_expand_builtin (exp, target, subtarget, mode, ignore)
   tree arg0, arg1, arg2, arg3;
   rtx op0, op1, op2, pat;
   enum machine_mode tmode, mode0, mode1, mode2;
-  int fcode = DECL_FUNCTION_CODE (fndecl);
+  unsigned int fcode = DECL_FUNCTION_CODE (fndecl);
 
   switch (fcode)
     {

diff --git a/gcc/expr.c b/gcc/expr.c
index 5a396f14f6b32e5568b1156e98f007a4851afb55..49b1c3e7e097f10f81506494af5f2fdf6ef1d448 100644 (file)
--- a/gcc/expr.c
+++ b/gcc/expr.c
@@ -1,6 +1,6 @@
 /* Convert tree expression to rtl instructions, for GNU compiler.
-   Copyright (C) 1988, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000
-   Free Software Foundation, Inc.
+   Copyright (C) 1988, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
+   2000, 2001 Free Software Foundation, Inc.
 
 This file is part of GNU CC.
 
@@ -212,7 +212,7 @@ static char direct_store[NUM_MACHINE_MODES];
    to perform a structure copy.  */
 #ifndef MOVE_BY_PIECES_P
 #define MOVE_BY_PIECES_P(SIZE, ALIGN) \
-  (move_by_pieces_ninsns (SIZE, ALIGN) < MOVE_RATIO)
+  (move_by_pieces_ninsns (SIZE, ALIGN) < (unsigned int) MOVE_RATIO)
 #endif
 
 /* This array records the insn_code of insns to perform block moves.  */
@@ -1967,7 +1967,7 @@ emit_group_load (dst, orig_src, ssize, align)
       int shift = 0;
 
       /* Handle trailing fragments that run over the size of the struct.  */
-      if (ssize >= 0 && bytepos + bytelen > ssize)
+      if (ssize >= 0 && bytepos + (HOST_WIDE_INT) bytelen > ssize)
        {
          shift = (bytelen - (ssize - bytepos)) * BITS_PER_UNIT;
          bytelen = ssize - bytepos;
@@ -1992,7 +1992,7 @@ emit_group_load (dst, orig_src, ssize, align)
          if (bytepos == 0
              && bytelen == GET_MODE_SIZE (GET_MODE (XEXP (src, 0))))
            tmps[i] = XEXP (src, 0);
-         else if (bytepos == GET_MODE_SIZE (GET_MODE (XEXP (src, 0)))
+         else if (bytepos == (HOST_WIDE_INT) GET_MODE_SIZE (GET_MODE (XEXP (src, 0)))
                   && bytelen == GET_MODE_SIZE (GET_MODE (XEXP (src, 1))))
            tmps[i] = XEXP (src, 1);
          else
@@ -2090,7 +2090,7 @@ emit_group_store (orig_dst, src, ssize, align)
       unsigned int bytelen = GET_MODE_SIZE (mode);
 
       /* Handle trailing fragments that run over the size of the struct.  */
-      if (ssize >= 0 && bytepos + bytelen > ssize)
+      if (ssize >= 0 && bytepos + (HOST_WIDE_INT) bytelen > ssize)
        {
          if (BYTES_BIG_ENDIAN)
            {
@@ -2539,7 +2539,7 @@ clear_storage (object, size, align)
      just move a zero.  Otherwise, do this a piece at a time.  */
   if (GET_MODE (object) != BLKmode
       && GET_CODE (size) == CONST_INT
-      && GET_MODE_SIZE (GET_MODE (object)) == INTVAL (size))
+      && GET_MODE_SIZE (GET_MODE (object)) == (unsigned int) INTVAL (size))
     emit_move_insn (object, CONST0_RTX (GET_MODE (object)));
   else
     {
@@ -4151,9 +4151,10 @@ store_expr (exp, target, want_value)
                {
                  addr = plus_constant (addr, TREE_STRING_LENGTH (exp));
                  size = plus_constant (size, -TREE_STRING_LENGTH (exp));
-                 align = MIN (align, (BITS_PER_UNIT
-                                      * (INTVAL (copy_size_rtx)
-                                         & - INTVAL (copy_size_rtx))));
+                 align = MIN (align,
+                              (unsigned int) (BITS_PER_UNIT
+                                              * (INTVAL (copy_size_rtx)
+                                                 & - INTVAL (copy_size_rtx))));
                }
              else
                {
@@ -4343,7 +4344,7 @@ store_constructor_field (target, bitsize, bitpos,
       /* Show the alignment may no longer be what it was and update the alias
         set, if required.  */
       if (bitpos != 0)
-       align = MIN (align, bitpos & - bitpos);
+       align = MIN (align, (unsigned int) bitpos & - bitpos);
       if (GET_CODE (target) == MEM)
        MEM_ALIAS_SET (target) = alias_set;
 
@@ -4429,7 +4430,7 @@ store_constructor (exp, target, align, cleared, size)
                    != fields_length (type))
                   || mostly_zeros_p (exp))
               && (GET_CODE (target) != REG
-                  || GET_MODE_SIZE (GET_MODE (target)) == size))
+                  || (HOST_WIDE_INT) GET_MODE_SIZE (GET_MODE (target)) == size))
        {
          if (! cleared)
            clear_storage (target, GEN_INT (size), align);
@@ -5040,7 +5041,7 @@ store_field (target, bitsize, bitpos, mode, exp, value_mode,
 
       PUT_MODE (blk_object, BLKmode);
 
-      if (bitsize != GET_MODE_BITSIZE (GET_MODE (target)))
+      if (bitsize != (HOST_WIDE_INT) GET_MODE_BITSIZE (GET_MODE (target)))
        emit_move_insn (object, target);
 
       store_field (blk_object, bitsize, bitpos, mode, exp, VOIDmode, 0,
@@ -7287,7 +7288,7 @@ expand_expr (exp, target, tmode, modifier)
                         MIN ((int_size_in_bytes (TREE_TYPE
                                                  (TREE_OPERAND (exp, 0)))
                               * BITS_PER_UNIT),
-                             GET_MODE_BITSIZE (mode)),
+                             (HOST_WIDE_INT) GET_MODE_BITSIZE (mode)),
                         0, TYPE_MODE (valtype), TREE_OPERAND (exp, 0),
                         VOIDmode, 0, BITS_PER_UNIT,
                         int_size_in_bytes (type), 0);