Fix most compile time warning messages
authorNick Clifton <nickc@redhat.com>
Tue, 8 Aug 2000 01:17:06 +0000 (01:17 +0000)
committerNick Clifton <nickc@gcc.gnu.org>
Tue, 8 Aug 2000 01:17:06 +0000 (01:17 +0000)
From-SVN: r35559

gcc/ChangeLog
gcc/config/mips/mips-protos.h
gcc/config/mips/mips.c

index 53a66e1e12b96a0fbce6f6bb8073223ef3ec2504..12609e2dcb62e3e8f3a4a9a4f92d2ac3ee5060dd 100644 (file)
@@ -1,4 +1,7 @@
-2000-08-07  Nick Clifton  <nickc@cygnus.com>
+2000-08-07  Nick Clifton  <nickc@redhat.com>
+
+       * config/mips/mips.c: Fix compile time warning messages.
+       * config/mips/mips-protos.h: Add prototype for equality_op.
 
        * mn10300.h (TARGET_SWITCHES): Document `-mam33'.
 
index 6f9f59b221f4122246d6b4497d3a8bcf0baa2504..126ca1d3933068e91cf207cea0a97a505d444604 100644 (file)
@@ -96,6 +96,7 @@ extern int            reg_or_0_operand PARAMS ((rtx, enum machine_mode));
 extern int             true_reg_or_0_operand PARAMS ((rtx, enum machine_mode));
 extern int             simple_memory_operand PARAMS ((rtx, enum machine_mode));
 extern int             double_memory_operand PARAMS ((rtx, enum machine_mode));
+extern int             equality_op PARAMS ((rtx, enum machine_mode));
 extern int             small_int PARAMS ((rtx, enum machine_mode));
 extern int             uns_arith_operand PARAMS ((rtx, enum machine_mode));
 extern struct rtx_def *        embedded_pic_offset PARAMS ((rtx));
index 85e28c53cbb2c9801248eaf70b39d851da012ae5..963a5ae3e035e23e4e97c7a7aaf8540165276c86 100644 (file)
@@ -692,7 +692,7 @@ mips16_simple_memory_operand (reg, offset, mode)
     off = 0x100;
   else
     off = 0x20;
-  if (INTVAL (offset) >= 0 && INTVAL (offset) < off * size)
+  if (INTVAL (offset) >= 0 && INTVAL (offset) < (HOST_WIDE_INT)(off * size))
     return 1;
   return 0;
 }
@@ -715,7 +715,7 @@ simple_memory_operand (op, mode)
   /* ??? This isn't strictly correct.  It is OK to accept multiword modes
      here, since the length attributes are being set correctly, but only
      if the address is offsettable.  LO_SUM is not offsettable.  */
-  if (GET_MODE_SIZE (GET_MODE (op)) > UNITS_PER_WORD)
+  if (GET_MODE_SIZE (GET_MODE (op)) > (unsigned) UNITS_PER_WORD)
     return 0;
 
   /* Decode the address now.  */
@@ -868,7 +868,7 @@ double_memory_operand (op, mode)
             address will get reloaded anyhow.  */
          if (GET_CODE (addr) == PLUS
              && GET_CODE (XEXP (addr, 0)) == REG
-             && (REGNO (XEXP (addr, 0)) == HARD_FRAME_POINTER_REGNUM
+             && (REGNO (XEXP (addr, 0)) == (unsigned) HARD_FRAME_POINTER_REGNUM
                  || REGNO (XEXP (addr, 0)) == STACK_POINTER_REGNUM)
              && ((GET_CODE (XEXP (addr, 1)) == CONST_INT
                   && ! SMALL_INT (XEXP (addr, 1)))
@@ -885,7 +885,7 @@ double_memory_operand (op, mode)
              maddr = XEXP (addr, 0);
              if (GET_CODE (maddr) == PLUS
                  && GET_CODE (XEXP (maddr, 0)) == REG
-                 && (REGNO (XEXP (maddr, 0)) == HARD_FRAME_POINTER_REGNUM
+                 && (REGNO (XEXP (maddr, 0)) == (unsigned) HARD_FRAME_POINTER_REGNUM
                      || REGNO (XEXP (maddr, 0)) == STACK_POINTER_REGNUM)
                  && ((GET_CODE (XEXP (maddr, 1)) == CONST_INT
                       && ! SMALL_INT (XEXP (maddr, 1)))
@@ -905,7 +905,7 @@ double_memory_operand (op, mode)
              addr = XEXP (XEXP (addr, 0), 0);
              if (GET_CODE (addr) == PLUS
                  && GET_CODE (XEXP (addr, 0)) == REG
-                 && (REGNO (XEXP (addr, 0)) == HARD_FRAME_POINTER_REGNUM
+                 && (REGNO (XEXP (addr, 0)) == (unsigned) HARD_FRAME_POINTER_REGNUM
                      || REGNO (XEXP (addr, 0)) == STACK_POINTER_REGNUM)
                  && ((GET_CODE (XEXP (addr, 1)) == CONST_INT
                       && ! SMALL_INT (XEXP (addr, 1)))
@@ -1199,7 +1199,7 @@ mips_check_split (address, mode)
 {     
   /* ??? This is the same check used in simple_memory_operand.
      We use it here because LO_SUM is not offsettable.  */
-  if (GET_MODE_SIZE (mode) > UNITS_PER_WORD)
+  if (GET_MODE_SIZE (mode) > (unsigned) UNITS_PER_WORD)
     return 0;
 
   if ((GET_CODE (address) == SYMBOL_REF && ! SYMBOL_REF_FLAG (address))
@@ -1246,7 +1246,7 @@ mips_legitimate_address_p (mode, xinsn, strict)
   /* Check for constant before stripping off SUBREG, so that we don't  
      accept (subreg (const_int)) which will fail to reload. */         
   if (CONSTANT_ADDRESS_P (xinsn)                                       
-      && ! (mips_split_addresses && mips_check_split (xinsn, mode))    
+      && ! (mips_split_addresses && mips_check_split (xinsn, mode))
       && (! TARGET_MIPS16 || mips16_constant (xinsn, mode, 1, 0)))     
     return 1;                                                          
                                                                        
@@ -1619,7 +1619,7 @@ mips_fill_delay_slot (ret, type, operands, cur_insn)
   dslots_number_nops = num_nops;
   mips_load_reg = set_reg;
   if (GET_MODE_SIZE (mode)
-      > (FP_REG_P (REGNO (set_reg)) ? UNITS_PER_FPREG : UNITS_PER_WORD))
+      > (unsigned) (FP_REG_P (REGNO (set_reg)) ? UNITS_PER_FPREG : UNITS_PER_WORD))
     mips_load_reg2 = gen_rtx_REG (SImode, REGNO (set_reg) + 1);
   else
     mips_load_reg2 = 0;
@@ -3183,7 +3183,7 @@ block_move_loop (dest_reg, src_reg, bytes, align, orig_dest, orig_src)
   rtx bytes_rtx;
   int leftover;
 
-  if (bytes < 2 * MAX_MOVE_BYTES)
+  if (bytes < 2U * MAX_MOVE_BYTES)
     abort ();
 
   leftover = bytes % MAX_MOVE_BYTES;
@@ -3250,7 +3250,7 @@ block_move_call (dest_reg, src_reg, bytes_rtx)
   /* We want to pass the size as Pmode, which will normally be SImode
      but will be DImode if we are using 64 bit longs and pointers.  */
   if (GET_MODE (bytes_rtx) != VOIDmode
-      && GET_MODE (bytes_rtx) != Pmode)
+      && GET_MODE (bytes_rtx) != (unsigned) Pmode)
     bytes_rtx = convert_to_mode (Pmode, bytes_rtx, 1);
 
 #ifdef TARGET_MEM_FUNCTIONS
@@ -3292,7 +3292,7 @@ expand_block_move (operands)
   if (constp && bytes == 0)
     return;
 
-  if (align > UNITS_PER_WORD)
+  if (align > (unsigned) UNITS_PER_WORD)
     align = UNITS_PER_WORD;
 
   /* Move the address into scratch registers.  */
@@ -3302,18 +3302,18 @@ expand_block_move (operands)
   if (TARGET_MEMCPY)
     block_move_call (dest_reg, src_reg, bytes_rtx);
 
-  else if (constp && bytes <= 2 * MAX_MOVE_BYTES
-          && align == UNITS_PER_WORD)
+  else if (constp && bytes <= 2U * MAX_MOVE_BYTES
+          && align == (unsigned) UNITS_PER_WORD)
     move_by_pieces (orig_dest, orig_src, bytes, align * BITS_PER_WORD);
        
-  else if (constp && bytes <= 2 * MAX_MOVE_BYTES)
+  else if (constp && bytes <= 2U * MAX_MOVE_BYTES)
     emit_insn (gen_movstrsi_internal (change_address (orig_dest, BLKmode,
                                                      dest_reg),
                                      change_address (orig_src, BLKmode,
                                                      src_reg),
                                      bytes_rtx, align_rtx));
 
-  else if (constp && align >= UNITS_PER_WORD && optimize)
+  else if (constp && align >= (unsigned) UNITS_PER_WORD && optimize)
     block_move_loop (dest_reg, src_reg, bytes, align, orig_dest, orig_src);
 
   else if (constp && optimize)
@@ -3857,7 +3857,7 @@ function_arg (cum, mode, type, named)
   rtx ret;
   int regbase = -1;
   int bias = 0;
-  int *arg_words = &cum->arg_words;
+  unsigned int *arg_words = &cum->arg_words;
   int struct_p = (type != 0
                  && (TREE_CODE (type) == RECORD_TYPE
                      || TREE_CODE (type) == UNION_TYPE
@@ -3938,7 +3938,7 @@ function_arg (cum, mode, type, named)
 
       /* Drops through.  */
     case BLKmode:
-      if (type != NULL_TREE && TYPE_ALIGN (type) > BITS_PER_WORD
+      if (type != NULL_TREE && TYPE_ALIGN (type) > (unsigned) BITS_PER_WORD
          && ! TARGET_64BIT && mips_abi != ABI_EABI)
        cum->arg_words += (cum->arg_words & 1);
       regbase = GP_ARG_FIRST;
@@ -3957,7 +3957,7 @@ function_arg (cum, mode, type, named)
       regbase = GP_ARG_FIRST;
     }
 
-  if (*arg_words >= MAX_ARGS_IN_REGISTERS)
+  if (*arg_words >= (unsigned) MAX_ARGS_IN_REGISTERS)
     {
       if (TARGET_DEBUG_E_MODE)
        fprintf (stderr, "<stack>%s\n", struct_p ? ", [struct]" : "");
@@ -4007,7 +4007,7 @@ function_arg (cum, mode, type, named)
 
              chunks
                = tree_low_cst (TYPE_SIZE_UNIT (type), 1) / UNITS_PER_WORD;
-             if (chunks + *arg_words + bias > MAX_ARGS_IN_REGISTERS)
+             if (chunks + *arg_words + bias > (unsigned) MAX_ARGS_IN_REGISTERS)
                chunks = MAX_ARGS_IN_REGISTERS - *arg_words - bias;
 
              /* assign_parms checks the mode of ENTRY_PARM, so we must
@@ -4110,7 +4110,7 @@ function_arg_partial_nregs (cum, mode, type, named)
   if ((mode == BLKmode
        || GET_MODE_CLASS (mode) != MODE_COMPLEX_INT
        || GET_MODE_CLASS (mode) != MODE_COMPLEX_FLOAT)
-      && cum->arg_words < MAX_ARGS_IN_REGISTERS
+      && cum->arg_words < (unsigned) MAX_ARGS_IN_REGISTERS
       && mips_abi != ABI_EABI)
     {
       int words;
@@ -4120,7 +4120,7 @@ function_arg_partial_nregs (cum, mode, type, named)
       else
        words = (GET_MODE_SIZE (mode) + UNITS_PER_WORD - 1) / UNITS_PER_WORD;
 
-      if (words + cum->arg_words <= MAX_ARGS_IN_REGISTERS)
+      if (words + cum->arg_words <= (unsigned) MAX_ARGS_IN_REGISTERS)
        return 0;               /* structure fits in registers */
 
       if (TARGET_DEBUG_E_MODE)
@@ -4130,7 +4130,7 @@ function_arg_partial_nregs (cum, mode, type, named)
       return MAX_ARGS_IN_REGISTERS - cum->arg_words;
     }
 
-  else if (mode == DImode && cum->arg_words == MAX_ARGS_IN_REGISTERS-1
+  else if (mode == DImode && cum->arg_words == MAX_ARGS_IN_REGISTERS - 1U
           && ! TARGET_64BIT && mips_abi != ABI_EABI)
     {
       if (TARGET_DEBUG_E_MODE)
@@ -4638,7 +4638,7 @@ override_options ()
      user. */
 #ifdef TARGET_DEFAULT
   if (TARGET_SINGLE_FLOAT && TARGET_SOFT_FLOAT)
-    target_flags &= ~(TARGET_DEFAULT&(MASK_SOFT_FLOAT|MASK_SINGLE_FLOAT));
+    target_flags &= ~((TARGET_DEFAULT) & (MASK_SOFT_FLOAT | MASK_SINGLE_FLOAT));
 #endif
 
   /* Get the architectural level.  */
@@ -4755,10 +4755,10 @@ override_options ()
     mips_cpu_string = MIPS_CPU_STRING_DEFAULT;
 #endif
 
-  /* Identify the processor type */
+  /* Identify the processor type */
   if (mips_cpu_string == 0
-      || !strcmp (mips_cpu_string, "default")
-      || !strcmp (mips_cpu_string, "DEFAULT"))
+      || ! strcmp (mips_cpu_string, "default")
+      || ! strcmp (mips_cpu_string, "DEFAULT"))
     {
       switch (mips_isa)
        {
@@ -4780,7 +4780,6 @@ override_options ()
          break;
        }
     }
-
   else
     {
       const char *p = mips_cpu_string;
@@ -4795,7 +4794,7 @@ override_options ()
        p++;
 
       /* Since there is no difference between a R2000 and R3000 in
-        terms of the scheduler, we collapse them into just an R3000. */
+        terms of the scheduler, we collapse them into just an R3000.  */
 
       mips_cpu = PROCESSOR_DEFAULT;
       switch (*p)
@@ -6419,7 +6418,7 @@ save_restore_insns (store_p, large_reg, large_offset, file)
                  && GET_MODE (base_reg_rtx) == SImode)
                {
                  insn = emit_move_insn (base_reg_rtx,
-                                        GEN_INT (gp_offset & 0xffff0000));
+                                        GEN_INT (gp_offset & 0xffff0000U));
                  if (store_p)
                    RTX_FRAME_RELATED_P (insn) = 1;
                  insn
@@ -6637,7 +6636,7 @@ save_restore_insns (store_p, large_reg, large_offset, file)
                  && GET_MODE (base_reg_rtx) == SImode)
                {
                  insn = emit_move_insn (base_reg_rtx,
-                                        GEN_INT (fp_offset & 0xffff0000));
+                                        GEN_INT (fp_offset & 0xffff0000U));
                  if (store_p)
                    RTX_FRAME_RELATED_P (insn) = 1;
                  insn = emit_insn (gen_iorsi3 (base_reg_rtx, base_reg_rtx,
@@ -6839,9 +6838,9 @@ function_prologue (file, size)
          dest = SET_DEST (set);
          if (GET_CODE (dest) != MEM)
            continue;
-         if (GET_MODE_SIZE (GET_MODE (dest)) == UNITS_PER_WORD)
+         if (GET_MODE_SIZE (GET_MODE (dest)) == (unsigned) UNITS_PER_WORD)
            ;
-         else if (GET_MODE_SIZE (GET_MODE (dest)) == 2 * UNITS_PER_WORD
+         else if (GET_MODE_SIZE (GET_MODE (dest)) == 2U * UNITS_PER_WORD
                   && REGNO (src) < GP_REG_FIRST + 7)
            ;
          else
@@ -6851,11 +6850,12 @@ function_prologue (file, size)
          if (GET_CODE (base) != REG
              || GET_CODE (offset) != CONST_INT)
            continue;
-         if (REGNO (base) == STACK_POINTER_REGNUM
-             && INTVAL (offset) == tsize + (REGNO (src) - 4) * UNITS_PER_WORD)
+         if (REGNO (base) == (unsigned) STACK_POINTER_REGNUM
+             && (unsigned HOST_WIDE_INT) INTVAL (offset)
+             == tsize + (REGNO (src) - 4) * UNITS_PER_WORD)
            ;
-         else if (REGNO (base) == HARD_FRAME_POINTER_REGNUM
-                  && (INTVAL (offset)
+         else if (REGNO (base) == (unsigned) HARD_FRAME_POINTER_REGNUM
+                  && ((unsigned HOST_WIDE_INT) INTVAL (offset)
                       == (tsize
                           + (REGNO (src) - 4) * UNITS_PER_WORD
                           - current_function_outgoing_args_size)))
@@ -6966,7 +6966,7 @@ mips_expand_prologue ()
      of the first argument in the variable part of the argument list,
      otherwise GP_ARG_LAST+1.  Note also if the last argument is 
      the varargs special argument, and treat it as part of the
-     variable arguments. 
+     variable arguments.
      
      This is only needed if store_args_on_stack is true. */
 
@@ -7179,7 +7179,7 @@ mips_expand_prologue ()
                  && GET_MODE (tmp_rtx) == SImode)
                {
                  insn = emit_move_insn (tmp_rtx,
-                                        GEN_INT (tsize & 0xffff0000));
+                                        GEN_INT (tsize & 0xffff0000U));
                  RTX_FRAME_RELATED_P (insn) = 1;
                  insn = emit_insn (gen_iorsi3 (tmp_rtx, tmp_rtx,
                                                GEN_INT (tsize & 0x0000ffff)));
@@ -7298,7 +7298,7 @@ mips_expand_prologue ()
 /* Do any necessary cleanup after a function to restore stack, frame,
    and regs. */
 
-#define RA_MASK ((long) 0x80000000)    /* 1 << 31 */
+#define RA_MASK ((unsigned long) 0x80000000U)  /* 1 << 31 */
 #define PIC_OFFSET_TABLE_MASK (1 << (PIC_OFFSET_TABLE_REGNUM - GP_REG_FIRST))
 
 void
@@ -7580,7 +7580,7 @@ mips_select_rtx_section (mode, x)
       /* For hosted applications, always put constants in small data if
         possible, as this gives the best performance.  */
      
-      if (GET_MODE_SIZE (mode) <= mips_section_threshold
+      if (GET_MODE_SIZE (mode) <= (unsigned) mips_section_threshold
          && mips_section_threshold > 0)
        SMALL_DATA_SECTION ();
       else if (flag_pic && symbolic_expression_p (x))
@@ -7791,16 +7791,15 @@ function_arg_pass_by_reference (cum, mode, type, named)
      here hopefully is not relevant to mips_va_arg.  */
   if (cum && MUST_PASS_IN_STACK (mode, type))
      {
-        /* Don't pass the actual CUM to FUNCTION_ARG, because we would 
-           get double copies of any offsets generated for small structs 
-           passed in registers. */
-        CUMULATIVE_ARGS temp;
-        temp = *cum;
-        if (FUNCTION_ARG (temp, mode, type, named) != 0)
-           return 1;
+       /* Don't pass the actual CUM to FUNCTION_ARG, because we would 
+         get double copies of any offsets generated for small structs 
+         passed in registers. */
+       CUMULATIVE_ARGS temp;
+       temp = *cum;
+       if (FUNCTION_ARG (temp, mode, type, named) != 0)
+        return 1;
      }
 
-
   /* Otherwise, we only do this if EABI is selected.  */
   if (mips_abi != ABI_EABI)
     return 0;
@@ -8145,7 +8144,10 @@ mips16_constant (x, mode, addr, addend)
          knows how to handle this.  We can always accept a string
          constant, which is the other case in which SYMBOL_REF_FLAG
          will be set.  */
-      if (! addr && ! addend && SYMBOL_REF_FLAG (x) && mode == Pmode)
+      if (! addr
+         && ! addend
+         && SYMBOL_REF_FLAG (x)
+         && mode == (enum machine_mode) Pmode)
        return 1;
 
       /* We can accept a string constant, which will have
@@ -8716,14 +8718,14 @@ mips16_optimize_gp (first)
          && GET_CODE (XEXP (SET_SRC (set), 0)) == REG
          && REGNO (XEXP (SET_SRC (set), 0)) == GP_REG_FIRST + 28
          && GET_CODE (SET_DEST (set)) == REG
-         && GET_MODE (SET_DEST (set)) == Pmode)
+         && GET_MODE (SET_DEST (set)) == (unsigned) Pmode)
        gpcopy = SET_DEST (set);
       else if (slot == NULL_RTX
               && gpcopy != NULL_RTX
               && GET_CODE (SET_DEST (set)) == MEM
               && GET_CODE (SET_SRC (set)) == REG
               && REGNO (SET_SRC (set)) == REGNO (gpcopy)
-              && GET_MODE (SET_DEST (set)) == Pmode)
+              && GET_MODE (SET_DEST (set)) == (unsigned) Pmode)
        {
          rtx base, offset;
 
@@ -8740,7 +8742,7 @@ mips16_optimize_gp (first)
               && reg_overlap_mentioned_p (SET_DEST (set), gpcopy)
               && (GET_CODE (SET_DEST (set)) != REG
                   || REGNO (SET_DEST (set)) != REGNO (gpcopy)
-                  || GET_MODE (SET_DEST (set)) != Pmode
+                  || GET_MODE (SET_DEST (set)) != (unsigned) Pmode
                   || ((GET_CODE (SET_SRC (set)) != CONST
                        || GET_CODE (XEXP (SET_SRC (set), 0)) != REG
                        || (REGNO (XEXP (SET_SRC (set), 0))
@@ -8859,7 +8861,7 @@ mips16_optimize_gp (first)
 
       set = PATTERN (insn);
       if (GET_CODE (set) != SET
-         || GET_MODE (SET_DEST (set)) != Pmode)
+         || GET_MODE (SET_DEST (set)) != (unsigned) Pmode)
        continue;
 
       if (GET_CODE (SET_DEST (set)) == MEM