re PR c/16046 (ICE tree check: expected tree_list, have identifier_node in…

re PR c/16046 (ICE tree check: expected tree_list, have identifier_node in decl_attributes, at attribs.c:151)

2004-09-10  Eric Christopher  <echristo@redhat.com>

	PR c/16046
	* c-parse.in (attribute): Add NULL_TREE for error case
	for error recovery.

2004-09-10  Eric Christopher  <echristo@redhat.com>

	* gcc.dg/20040910-1.c: New test.

From-SVN: r87330

gcc/ChangeLog

+2004-09-10  Eric Christopher  <echristo@redhat.com>
+
+	PR c/16046
+	* c-parse.in (attribute): Add NULL_TREE for error case
+	for error recovery.
+
 2004-09-10  Zack Weinberg  <zack@codesourcery.com>
 
 	* c-common.h (struct c_fileinfo): Comment moved here from cp-tree.h.
@@ -6,8 +12,8 @@
 	* c-lex.c (get_fileinfo): Initialize file_info_tree here, if
 	it's not already been done ...
 	(init_c_lex): ... not here.
-	(fe_file_change): Don't call extract_interface_info
-.
+	(fe_file_change): Don't call extract_interface_info.
+
 2004-09-11  Joseph S. Myers  <jsm@polyomino.org.uk>
 
 	* c-tree.h (struct c_declspecs): Remove typedef_decl.  Add

gcc/c-parse.in

@@ -1359,7 +1359,7 @@ attribute:
                 '(' '(' attribute_list ')' ')' start_string_translation
 		{ $$ = $5; }
       | ATTRIBUTE error start_string_translation
-                {}
+                { $$ = NULL_TREE; }
 	;
 
 attribute_list:
@@ -2048,9 +2048,9 @@ c99_block_lineno_labeled_stmt:
 
 lineno_stmt:
 	  save_location stmt
-		{ 
+		{
 		  /* Two cases cannot and do not have line numbers associated:
-		     If stmt is degenerate, such as "2;", then stmt is an 
+		     If stmt is degenerate, such as "2;", then stmt is an
 		     INTEGER_CST, which cannot hold line numbers.  But that's
 		     ok because the statement will either be changed to a
 		     MODIFY_EXPR during gimplification of the statement expr,
@@ -2831,7 +2831,7 @@ optparms:
 						     grokparm ($3)));
 		}
 	;
-	
+
 optellipsis:
 	  /* NULL */
 		{

gcc/testsuite/ChangeLog

+2004-09-10  Eric Christopher  <echristo@redhat.com>
+
+	* gcc.dg/20040910-1.c: New test.
+
 2004-09-11  Joseph S. Myers  <jsm@polyomino.org.uk>
 
 	* gcc.dg/bitfld-9.c: New test.
@@ -45,7 +49,7 @@
 	* gcc.dg/darwin-ld-20040828-1.c: New test.
 	* gcc.dg/darwin-ld-20040828-2.c: New test.
 	* gcc.dg/darwin-ld-20040828-3.c: New test.
-	
+
 2004-09-08  Ziemowit Laski  <zlaski@apple.com>
 
 	* objc.dg/bitfield-2.m: Make 'id' definition a typedef.
@@ -66,10 +70,10 @@
 	LD_LIBRARYN32_PATH, LD_LIBRARY64_PATH and DYLD_LIBRARY_PATH.
 
 2004-09-06  Devang Patel  <dpatel@apple.com>
-	
+
 	* gcc.dg/tree-ssa/ifc-20040816-1.c: Use -ftree-vectorize.
 	* gcc.dg/tree-ssa/ifc-20040816-2.c: Use -ftree-vectorize.
-	
+
 2004-09-06  Zack Weinberg  <zack@codesourcery.com>
 
 	* g++.old-deja/g++.other/enum2.C: Move dg-error markers to

gcc/testsuite/gcc.dg/20040910-1.c

+/* Tests error recovery for invalid code.  */
+__attribute__((foo)  int f (){} /* { dg-error "syntax error before \"int\"" } */