diff options
| author | Mike Stump <mrs@apple.com> | 2006-08-15 18:01:19 +0000 |
|---|---|---|
| committer | Mike Stump <mrs@gcc.gnu.org> | 2006-08-15 18:01:19 +0000 |
| commit | 4adb785d4481822edd5350f38b3fd2d3bcbfddca (patch) | |
| tree | c67b6c012b86ed35cb28402e0f422833b7295e5f /gcc/c-parser.c | |
| parent | d47e3adf1d1100661aa25c0668575c649a690b4a (diff) | |
| download | gcc-4adb785d4481822edd5350f38b3fd2d3bcbfddca.zip gcc-4adb785d4481822edd5350f38b3fd2d3bcbfddca.tar.gz gcc-4adb785d4481822edd5350f38b3fd2d3bcbfddca.tar.bz2 | |
re PR c/28280 (bogus "statement with no effect" warning with VLA and typeof)
PR c/28280
* c-parser.c (c_parser_typeof_specifier): Don't use
c_finish_expr_stmt, open code desired semantics instead.
From-SVN: r116163
Diffstat (limited to 'gcc/c-parser.c')
| -rw-r--r-- | gcc/c-parser.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/gcc/c-parser.c b/gcc/c-parser.c index 848da63..6e243df 100644 --- a/gcc/c-parser.c +++ b/gcc/c-parser.c @@ -2164,7 +2164,19 @@ c_parser_typeof_specifier (c_parser *parser) is evaluated, this can be evaluated. For now, we avoid evaluation when the context might. */ if (!skip_evaluation && was_vm) - c_finish_expr_stmt (expr.value); + { + tree e = expr.value; + + /* If the expression is not of a type to which we cannot assign a line + number, wrap the thing in a no-op NOP_EXPR. */ + if (DECL_P (e) || CONSTANT_CLASS_P (e)) + e = build1 (NOP_EXPR, void_type_node, e); + + if (EXPR_P (e)) + SET_EXPR_LOCATION (e, input_location); + + add_stmt (e); + } pop_maybe_used (was_vm); } c_parser_skip_until_found (parser, CPP_CLOSE_PAREN, "expected %<)%>"); |