[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163


pinskia@physics.uc.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4                         |3.3.1

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163


pinskia at physics dot uc dot edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.1                       |3.3.2


------- Additional Comments From pinskia at physics dot uc dot edu  2003-06-22 01:39 -------
I am moving this to 3.3.2 since this does not show up in released versions of gcc 
(because of --disable-checking is defualt).

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163



------- Additional Comments From pinskia at physics dot uc dot edu  2003-07-23 19:51 -------
>From Phil's regressio hunter:
: Search converges between 2001-06-03-trunk (#22) and 2001-06-10-trunk (#23).

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163


pinskia at physics dot uc dot edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |minor

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.2                       |3.4


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-09-17 05:45 -------
I am moving this to 3.4 as no released compiler will see as checking is disabled by default in them.

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[dhazeghi at yahoo dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163


dhazeghi at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|2003-06-23 11:16:26         |2003-10-24 17:47:42
               date|                            |


------- Additional Comments From dhazeghi at yahoo dot com  2003-10-24 17:47 -------
Still present on mainline.

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-22 06:18 -------
I posted a fix but it is not up there yet.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-22 16:07 -------
Since my email has not come through yet (why?):
ChangeLog:

	* c-decl.c (poplevel): Only set DECL_INITIAL of a current function
	if it is non-null.
	(finish_function): Check for error_mark_node on DECL_RESULT and DECL_RESULT
	of fndecl.
	(c_expand_body): Only expand when DECL_INITIAL of fndecl is not
	error_mark_node.


Patch:
Index: c-decl.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/c-decl.c,v
retrieving revision 1.462
diff -u -p -r1.462 c-decl.c
--- c-decl.c	21 Dec 2003 14:08:32 -0000	1.462
+++ c-decl.c	22 Dec 2003 06:04:20 -0000
@@ -676,7 +676,7 @@ poplevel (int keep, int dummy ATTRIBUTE_
     IDENTIFIER_TAG_VALUE (TREE_PURPOSE (p)) = TREE_VALUE (p);

   /* Dispose of the block that we just made inside some higher level.  */
-  if (scope->function_body)
+  if (scope->function_body && current_function_decl)
     DECL_INITIAL (current_function_decl) = block;
   else if (scope->outer)
     {
@@ -6088,11 +6088,13 @@ finish_function (void)
  	}
     }

-  BLOCK_SUPERCONTEXT (DECL_INITIAL (fndecl)) = fndecl;
+  if (DECL_INITIAL (fndecl) != error_mark_node)
+    BLOCK_SUPERCONTEXT (DECL_INITIAL (fndecl)) = fndecl;

   /* Must mark the RESULT_DECL as being in this function.  */

-  DECL_CONTEXT (DECL_RESULT (fndecl)) = fndecl;
+  if (DECL_RESULT (fndecl) != error_mark_node)
+    DECL_CONTEXT (DECL_RESULT (fndecl)) = fndecl;

   if (MAIN_NAME_P (DECL_NAME (fndecl)) && flag_hosted)
     {
@@ -6223,7 +6225,9 @@ c_expand_body_1 (tree fndecl, int nested
 void
 c_expand_body (tree fndecl)
 {
-  c_expand_body_1 (fndecl, 0);
+
+  if (DECL_INITIAL (fndecl) != error_mark_node)
+    c_expand_body_1 (fndecl, 0);
 }
 
\f
 /* Check the declarations given in a for-loop for satisfying the C99


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-22 16:22 -------
Finnaly was able to send one out: <http://gcc.gnu.org/ml/gcc-patches/2003-12/
msg01896.html>.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2003-12-22 18:17 -------
Subject: Bug 9163

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	pinskia@gcc.gnu.org	2003-12-22 18:16:58

Modified files:
	gcc            : ChangeLog c-decl.c 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gcc.dg: 20031222-1.c 

Log message:
	2003-12-22  Andrew Pinski  <pinskia@physics.uc.edu>
	
	PR c/9163
	* c-decl.c (poplevel): Only set DECL_INITIAL of a current function
	if it is non-null.
	(finish_function): Check for error_mark_node or null on DECL_RESULT and
	DECL_RESULT of fndecl.
	(c_expand_body): Only expand when DECL_INITIAL of fndecl is not
	error_mark_node and not null.
	
	2003-12-22  Andrew Pinski  <pinskia@physics.uc.edu>
	
	PR c/9163
	* gcc.dg/20031222-1.c: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.2071&r2=2.2072
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/c-decl.c.diff?cvsroot=gcc&r1=1.462&r2=1.463
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.3296&r2=1.3297
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/20031222-1.c.diff?cvsroot=gcc&r1=NONE&r2=1.1



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163

[Bug c/9163] [3.3/3.4 regression] ICE in genrtl_compound_stmt at c-semantics.c:776 with c99 mode and checking enabled

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-22 18:17 -------
Fixed for 3.4.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9163