[Bug debug/12389] New: ICE in gen_subprogram_die for nested function

[Bug debug/12389] New: ICE in gen_subprogram_die for nested function

[rsandifo 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=12389

           Summary: ICE in gen_subprogram_die for nested function
           Product: gcc
           Version: 3.4
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: debug
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rsandifo at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org,rsandifo at gcc dot gnu
                    dot org
GCC target triplet: i686-pc-linux-gnu

Sorry if this already a known problem!  I tried searching for it but
couldn't find anything.

The following test case:

int f1 (int y)
{
  int f2() { return y; }
  return f2();
}

causes an ICE on i686-pc-linux-gnu if compiled with -O2 -g:

/tmp/foo.c: In function `f2':

/tmp/foo.c:5: internal compiler error: in gen_subprogram_die, at dwarf2out.c:10614
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.

It works OK if compiled with -O3 -g or with -O2 -fno-unit-at-a-time -g.

At a guess, I think that dwarf code is getting confused by the inlining
of functions that aren't marked !DECL_INLINE.  It'll be a few days
before I can look into much more though.

[Bug debug/12389] ICE in gen_subprogram_die for nested function

[bangerth at dealii 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=12389


bangerth at dealii dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2003-09-24 13:58:54
               date|                            |


------- Additional Comments From bangerth at dealii dot org  2003-09-24 13:58 -------
Confirmed. A mainline regression

[Bug debug/12389] ICE in gen_subprogram_die for nested function

[rsandifo 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=12389



------- Additional Comments From rsandifo at gcc dot gnu dot org  2003-09-24 14:19 -------
Ooops... don't know what I was on when I wrote the final paragraph.
It should have read:

At a guess, I think that the dwarf code is getting confused by the inlining
of functions that aren't marked as DECL_INLINE.  It'll be a few days before
I can look into this in more detail though.

[Bug debug/12389] [3.4 Regresson] ICE in gen_subprogram_die for nested function

[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=12389


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |critical
           Keywords|                            |ice-on-valid-code
            Summary|ICE in gen_subprogram_die   |[3.4 Regresson] ICE in
                   |for nested function         |gen_subprogram_die for
                   |                            |nested function
   Target Milestone|---                         |3.4

[Bug debug/12389] [3.4 Regresson] ICE in gen_subprogram_die for nested function

[cgd at broadcom 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=12389



------- Additional Comments From cgd at broadcom dot com  2003-10-17 03:57 -------
I saw this on mips64-linux native, compiling glibc.  I tried the attached
test case and on mips64-linux it doesn't show the problem.

This test case does:

int x (char *s)
{
  int y () { return (strlen (s)); }
  return y (s);
}

with gcc -g -O2 -c src.c, compiled from unmodified sources as of 2003-10-15
00:00 UTC.

When this is fixed, and testcases are added, IMO would be good to have both
added.

[Bug debug/12389] [3.4 Regresson] ICE in gen_subprogram_die for nested function

[cvs-commit 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=12389



------- Additional Comments From cvs-commit at gcc dot gnu dot org  2003-10-22 11:58 -------
Subject: Bug 12389

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	hubicka@gcc.gnu.org	2003-10-22 11:58:10

Modified files:
	gcc            : ChangeLog Makefile.in c-decl.c cgraph.c 
	                 cgraph.h cgraphunit.c dwarf2out.c toplev.c 

Log message:
	PR debug/12389
	* Makefile.in (dwarf2out.o): Depend on cgraph.h.
	* cgraph.c (cgraph_function_possibly_inlined_p): New function.
	* cgraph.h (cgraph_function_possibly_inlined_p): Declare.
	(cgraph_global_info): Add flag inlined
	* dwarf2out.c (gen_subprogram_die, gen_decl_die): Use
	cgraph_function_possibly_inded_p
	* cgraphunit.c (mark_inline): Set inlined flag.
	* toplev.c (rest_of_decl_compilation): Call outlining_inline_function
	only for possibly inlined functions.
	* c-decl.c (duplicate_decls): Never output abstract DIE representing old
	body of function.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.1514&r2=2.1515
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/Makefile.in.diff?cvsroot=gcc&r1=1.1181&r2=1.1182
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/c-decl.c.diff?cvsroot=gcc&r1=1.454&r2=1.455
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cgraph.c.diff?cvsroot=gcc&r1=1.32&r2=1.33
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cgraph.h.diff?cvsroot=gcc&r1=1.22&r2=1.23
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cgraphunit.c.diff?cvsroot=gcc&r1=1.36&r2=1.37
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/dwarf2out.c.diff?cvsroot=gcc&r1=1.457&r2=1.458
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/toplev.c.diff?cvsroot=gcc&r1=1.836&r2=1.837

[Bug debug/12389] [3.4 Regresson] ICE in gen_subprogram_die for nested function

[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=12389


pinskia at gcc dot gnu dot org changed:

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


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

[Bug debug/12389] [3.4 Regresson] ICE in gen_subprogram_die for nested function

[cgd at broadcom 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=12389



------- Additional Comments From cgd at broadcom dot com  2003-10-24 18:54 -------
Err, the problem does seem to have fixed, but (given that apparently no
existing tests in the testsuite noticed the problem) wouldn't it have been
good to add some of the examples we provided?

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[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=12389


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |
            Summary|[3.4 Regresson] ICE in      |[3.4 Regresson] [testcase
                   |gen_subprogram_die for      |needed] ICE in
                   |nested function             |gen_subprogram_die for
                   |                            |nested function


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-10-25 01:03 -------
Reopening for the need for a testcase added to the testsuite.

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[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=12389


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|critical                    |normal
             Status|REOPENED                    |NEW

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |minor
   Last reconfirmed|2003-09-24 13:58:54         |2003-12-05 04:05:54
               date|                            |


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

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-18 11:47 -------
I will commit a testcase tommorrow with the other one which was caused by the patch 
which fixed this one :).

-- 
           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=12389

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[cvs-commit at gcc dot gnu dot org]

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

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

Modified files:
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gcc.dg: 20031218-1.c 20031218-2.c 20031218-3.c 

Log message:
	PR debug/12923
	* gcc.dg/20031218-1.c: New test.
	
	PR debug/12389
	* gcc.dg/20031218-2.c: New test.
	* gcc.dg/20031218-3.c: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.3284&r2=1.3285
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/20031218-1.c.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/20031218-2.c.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/20031218-3.c.diff?cvsroot=gcc&r1=NONE&r2=1.1



-- 


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

[Bug debug/12389] [3.4 Regresson] [testcase needed] ICE in gen_subprogram_die for nested function

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-18 22:20 -------
Testcases installed.

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


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