diff options
| author | jsm28 <jsm28@138bc75d-0d04-0410-961f-82ee72b054a4> | 2005-08-06 11:31:49 +0000 |
|---|---|---|
| committer | jsm28 <jsm28@138bc75d-0d04-0410-961f-82ee72b054a4> | 2005-08-06 11:31:49 +0000 |
| commit | 1c3d3e7cbc93a587d68e8e91bc7e6c07cf205840 (patch) | |
| tree | 50083463eef147d4518d2138f1a14ce9e8fb18b7 /gcc/testsuite | |
| parent | ca8cce9fc2a2367248ffe52d526ce0bfbb1ed71b (diff) | |
| download | gcc-1c3d3e7cbc93a587d68e8e91bc7e6c07cf205840.tar.gz | |
PR c/23113
* stmt.c (warn_if_unused_value): Check TREE_NO_WARNING at start.
Don't handle NOP_EXPR, CONVERT_EXPR and NON_LVALUE_EXPR
specially. Check for side effects only for COND_EXPR.
* c-typeck.c (c_finish_stmt_expr): Mark statement expression
return with TREE_NO_WARNING.
testsuite:
* gcc.dg/Wunused-value-1.c: New test.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@102805 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc/testsuite')
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/Wunused-value-1.c | 31 |
2 files changed, 36 insertions, 0 deletions
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 0097a9d2cfa..6e27aec9c24 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-08-06 Joseph S. Myers <joseph@codesourcery.com> + + PR c/23113 + * gcc.dg/Wunused-value-1.c: New test. + 2005-08-06 Richard Sandiford <richard@codesourcery.com> PR rtl-optimization/23233 diff --git a/gcc/testsuite/gcc.dg/Wunused-value-1.c b/gcc/testsuite/gcc.dg/Wunused-value-1.c new file mode 100644 index 00000000000..0fc7c364ba8 --- /dev/null +++ b/gcc/testsuite/gcc.dg/Wunused-value-1.c @@ -0,0 +1,31 @@ +/* Test -Wunused-value. Bug 23113. */ +/* Origin: Joseph Myers <joseph@codesourcery.com> */ +/* { dg-do compile } */ +/* { dg-options "-Wunused-value" } */ + +int f (void); +void g (void); +int *p; + +void +h (void) +{ + 1 + f (); /* { dg-warning "value computed is not used" } */ + f () + f (); /* { dg-warning "value computed is not used" } */ + f () + f (), f (); /* { dg-warning "value computed is not used" } */ + (char) f (); /* { dg-warning "value computed is not used" } */ + g (); + f (); + (void) f (); + *p++; /* { dg-warning "value computed is not used" } */ + ++*p; + (*p ? f() : 0); + ({ f(); }); + /* Statement expressions may be used in macro expansions which like + functions return values which may or may not be of use, so don't + warn for them but do warn inside them. */ + ({ f() + 1; }); + ({ f(); 0; }); + ({ f() + 1; 0; }); /* { dg-warning "value computed is not used" } */ + 1 + ({ f(); }); /* { dg-warning "value computed is not used" } */ +} |