aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--gcc/ChangeLog4
-rw-r--r--gcc/stmt.c8
2 files changed, 8 insertions, 4 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e9146bd..dccfdf8 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2000-12-05 Richard Henderson <rth@redhat.com>
+
+ * stmt.c (warn_if_unused_value): Move side effects test earlier.
+
2000-12-05 Alan Modra <alan@linuxcare.com.au>
* bb-reorder.c (reorder_basic_blocks): Do check for EH edges even
diff --git a/gcc/stmt.c b/gcc/stmt.c
index 06242ce..d4b047c 100644
--- a/gcc/stmt.c
+++ b/gcc/stmt.c
@@ -1983,6 +1983,10 @@ warn_if_unused_value (exp)
if (VOID_TYPE_P (TREE_TYPE (exp)))
return 0;
+ /* If this is an expression with side effects, don't warn. */
+ if (TREE_SIDE_EFFECTS (exp))
+ return 0;
+
switch (TREE_CODE (exp))
{
case PREINCREMENT_EXPR:
@@ -2058,10 +2062,6 @@ warn_if_unused_value (exp)
&& TREE_THIS_VOLATILE (exp))
return 0;
- /* If this is an expression with side effects, don't warn. */
- if (TREE_SIDE_EFFECTS (exp))
- return 0;
-
/* If this is an expression which has no operands, there is no value
to be unused. There are no such language-independent codes,
but front ends may define such. */