* stmt.c (warn_if_unused_value): Move side effects test earlier.

From-SVN: r38030

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e9146bd78a3ca8b0add444c6a41bdd3c73577f19..dccfdf84b900b187e060b0227aa75db27554734b 100644 (file)
--- 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 06242ce8e09237f328849cd2a4f9f99826471a0a..d4b047c9a9c658c257cd64516e16b4a36c2e8185 100644 (file)
--- 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.  */