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
This commit is contained in:
Joseph Myers 2009-02-18 00:38:28 +00:00 committed by Joseph Myers
parent afd1efee35
commit 5600f233ef
4 changed files with 25 additions and 1 deletions

View File

@ -1,3 +1,9 @@
2009-02-18 Joseph Myers <joseph@codesourcery.com>
PR c/35447
* c-parser.c (c_parser_compound_statement): Always enter and leave
a scope.
2009-02-17 John David Anglin <dave.anglin@nrc-cnrc.gc.ca> 2009-02-17 John David Anglin <dave.anglin@nrc-cnrc.gc.ca>
PR target/34587 PR target/34587

View File

@ -3307,7 +3307,13 @@ c_parser_compound_statement (c_parser *parser)
{ {
tree stmt; tree stmt;
if (!c_parser_require (parser, CPP_OPEN_BRACE, "expected %<{%>")) 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); stmt = c_begin_compound_stmt (true);
c_parser_compound_statement_nostart (parser); c_parser_compound_statement_nostart (parser);
return c_end_compound_stmt (stmt, true); return c_end_compound_stmt (stmt, true);

View File

@ -1,3 +1,8 @@
2009-02-18 Joseph Myers <joseph@codesourcery.com>
PR c/35447
* gcc.dg/noncompile/pr35447-1.c: New test.
2009-02-18 Jakub Jelinek <jakub@redhat.com> 2009-02-18 Jakub Jelinek <jakub@redhat.com>
PR tree-optimization/36922 PR tree-optimization/36922

View File

@ -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" } */