From 6f1225504f3c4124fd4ab85edd0db620b14264ac Mon Sep 17 00:00:00 2001 From: Richard Henderson Date: Tue, 14 Nov 2000 01:51:46 -0800 Subject: 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 --- gcc/stmt.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'gcc/stmt.c') 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. */ -- cgit v1.1