This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Warning patches installed
- To: gcc-patches at gcc dot gnu dot org
- Subject: Warning patches installed
- From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Date: Mon, 14 Aug 2000 16:57:16 -0400 (EDT)
I installed the following which fixes a few warning nits.
Bootstrapped on sparc-sun-solaris2.7 and i686-pc-linux-gnu.
Index: ChangeLog
===================================================================
RCS file: /cvs/gcc/egcs/gcc/ChangeLog,v
retrieving revision 1.7420
diff -u -p -r1.7420 ChangeLog
--- ChangeLog 2000/08/14 20:46:14 1.7420
+++ ChangeLog 2000/08/14 20:53:52
@@ -1,3 +1,15 @@
+2000-08-14 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
+
+ * cse.c (fold_rtx): Avoid empty body in an if-statement.
+
+ * doloop.c (doloop_iterations_max, doloop_modify): Avoid using the
+ `U' integer constant suffix.
+
+ * dwarf2out.c (add_subscript_info): Avoid empty body in an
+ else-statement.
+
+ * sparc/sol2.h (__enable_execute_stack): Prototype.
+
2000-08-14 David Edelsohn <edelsohn@gnu.org>
* collect2.c: Remove use of AIX import file.
Index: cse.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/cse.c,v
retrieving revision 1.149
diff -u -p -r1.149 cse.c
--- cse.c 2000/08/14 17:54:23 1.149
+++ cse.c 2000/08/14 20:53:58
@@ -3646,7 +3646,7 @@ fold_rtx (x, insn)
if (fmt[i] == 'E')
/* Don't try to fold inside of a vector of expressions.
Doing nothing is harmless. */
- ;
+ {;}
}
/* If a commutative operation, place a constant integer as the second
Index: doloop.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/doloop.c,v
retrieving revision 1.1
diff -u -p -r1.1 doloop.c
--- doloop.c 2000/07/30 23:57:55 1.1
+++ doloop.c 2000/08/14 20:53:58
@@ -197,7 +197,7 @@ doloop_iterations_max (loop_info, mode,
if (GET_CODE (max_value) == CONST_INT)
umax = INTVAL (max_value);
else
- umax = (2U << (GET_MODE_BITSIZE (mode) - 1)) - 1;
+ umax = ((unsigned)2 << (GET_MODE_BITSIZE (mode) - 1)) - 1;
n_iterations_max = umax - umin;
break;
@@ -212,12 +212,12 @@ doloop_iterations_max (loop_info, mode,
if (GET_CODE (min_value) == CONST_INT)
smin = INTVAL (min_value);
else
- smin = -(1U << (GET_MODE_BITSIZE (mode) - 1));
+ smin = -((unsigned)1 << (GET_MODE_BITSIZE (mode) - 1));
if (GET_CODE (max_value) == CONST_INT)
smax = INTVAL (max_value);
else
- smax = (1U << (GET_MODE_BITSIZE (mode) - 1)) - 1;
+ smax = ((unsigned)1 << (GET_MODE_BITSIZE (mode) - 1)) - 1;
n_iterations_max = smax - smin;
break;
@@ -230,7 +230,7 @@ doloop_iterations_max (loop_info, mode,
else
/* We need to conservatively assume that we might have the maximum
number of iterations without any additional knowledge. */
- n_iterations_max = (2U << (GET_MODE_BITSIZE (mode) - 1)) - 1;
+ n_iterations_max = ((unsigned)2 << (GET_MODE_BITSIZE (mode) - 1)) - 1;
break;
default:
@@ -241,8 +241,9 @@ doloop_iterations_max (loop_info, mode,
/* If we know that the iteration count is non-negative then adjust
n_iterations_max if it is so large that it appears negative. */
- if (nonneg && n_iterations_max > (1U << (GET_MODE_BITSIZE (mode) - 1)))
- n_iterations_max = (1U << (GET_MODE_BITSIZE (mode) - 1)) - 1;
+ if (nonneg
+ && n_iterations_max > ((unsigned)1 << (GET_MODE_BITSIZE (mode) - 1)))
+ n_iterations_max = ((unsigned)1 << (GET_MODE_BITSIZE (mode) - 1)) - 1;
return n_iterations_max;
}
@@ -451,7 +452,7 @@ doloop_modify (loop, iterations, iterati
/* Determine if the iteration counter will be non-negative.
Note that the maximum value loaded is iterations_max - 1. */
if ((unsigned HOST_WIDE_INT) INTVAL (iterations_max)
- <= (1U << (GET_MODE_BITSIZE (GET_MODE (counter_reg)) - 1)))
+ <= ((unsigned)1 << (GET_MODE_BITSIZE (GET_MODE (counter_reg)) - 1)))
nonneg = 1;
break;
Index: dwarf2out.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/dwarf2out.c,v
retrieving revision 1.195
diff -u -p -r1.195 dwarf2out.c
--- dwarf2out.c 2000/08/06 17:55:22 1.195
+++ dwarf2out.c 2000/08/14 20:54:07
@@ -7566,7 +7566,7 @@ add_subscript_info (type_die, type)
/* We have an array type with an unspecified length. The DWARF-2
spec does not say how to handle this; let's just leave out the
bounds. */
- ;
+ {;}
#ifndef MIPS_DEBUGGING_INFO
}
Index: config/sparc/sol2.h
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/sparc/sol2.h,v
retrieving revision 1.21
diff -u -p -r1.21 sol2.h
--- sol2.h 2000/03/13 17:34:57 1.21
+++ sol2.h 2000/08/14 20:54:07
@@ -255,6 +255,7 @@ static void check_enabling(void) \
need_enable_exec_stack = 1; \
} \
\
+extern void __enable_execute_stack (void *); \
void \
__enable_execute_stack (addr) \
void *addr; \