stmt.c (warn_if_unused_value): Don't warn if the expression has side effects.

        * stmt.c (warn_if_unused_value): Don't warn if the expression
        has side effects.

From-SVN: r37448

1 files changed, 4 insertions, 3 deletions

diff --git a/gcc/stmt.c b/gcc/stmt.c
index 4e42e69..82a390d 100644
--- a/gcc/stmt.c
+++ b/gcc/stmt.c
@@ -2004,9 +2004,6 @@ warn_if_unused_value (exp)
     case TRY_CATCH_EXPR:
     case WITH_CLEANUP_EXPR:
     case EXIT_EXPR:
-      /* We don't warn about COND_EXPR because it may be a useful
-	 construct if either arm contains a side effect.  */
-    case COND_EXPR:
       return 0;
 
     case BIND_EXPR:
@@ -2067,6 +2064,10 @@ 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.  */