From: Joseph Myers Date: Wed, 18 Feb 2009 00:38:28 +0000 (+0000) Subject: re PR c/35447 (ICE with broken statement expression) X-Git-Tag: releases/gcc-4.4.0~513 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5600f233ef5f740908f0480b76936e503b1d4cda;p=thirdparty%2Fgcc.git re PR c/35447 (ICE with broken statement expression) PR c/35447 * c-parser.c (c_parser_compound_statement): Always enter and leave a scope. testsuite: * gcc.dg/noncompile/pr35447-1.c: New test. From-SVN: r144256 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ca77fc468a3e..17869d7b11c0 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2009-02-18 Joseph Myers + + PR c/35447 + * c-parser.c (c_parser_compound_statement): Always enter and leave + a scope. + 2009-02-17 John David Anglin PR target/34587 diff --git a/gcc/c-parser.c b/gcc/c-parser.c index 9aef031a8eb6..6dfcb6015972 100644 --- a/gcc/c-parser.c +++ b/gcc/c-parser.c @@ -3307,7 +3307,13 @@ c_parser_compound_statement (c_parser *parser) { tree stmt; if (!c_parser_require (parser, CPP_OPEN_BRACE, "expected %<{%>")) - return error_mark_node; + { + /* Ensure a scope is entered and left anyway to avoid confusion + if we have just prepared to enter a function body. */ + stmt = c_begin_compound_stmt (true); + c_end_compound_stmt (stmt, true); + return error_mark_node; + } stmt = c_begin_compound_stmt (true); c_parser_compound_statement_nostart (parser); return c_end_compound_stmt (stmt, true); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 3cbc3e792290..055c2b19d12a 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2009-02-18 Joseph Myers + + PR c/35447 + * gcc.dg/noncompile/pr35447-1.c: New test. + 2009-02-18 Jakub Jelinek PR tree-optimization/36922 diff --git a/gcc/testsuite/gcc.dg/noncompile/pr35447-1.c b/gcc/testsuite/gcc.dg/noncompile/pr35447-1.c new file mode 100644 index 000000000000..9c31e9bb617c --- /dev/null +++ b/gcc/testsuite/gcc.dg/noncompile/pr35447-1.c @@ -0,0 +1,7 @@ +/* ICE arising from preparing to enter a scope for a nested function + but never doing so. PR 35447. */ + +void foo() +{ + ({ int i().; }); /* { dg-error "expected" } */ +} /* { dg-error "expected" } */