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

From-SVN: r38030

gcc/ChangeLog

+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

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.  */