[Bug c++/22003] New: -freorder-blocks-and-partition causes ICE on 4.1.0 (20050604)

[Bug c++/22003] New: -freorder-blocks-and-partition causes ICE on 4.1.0 (20050604)

[zlynx at acm dot org]

x86_64-pc-linux-gnu-g++ -ansi -march=athlon64 -O3 -g -pipe -funit-at-a-time
-fprefetch-loop-arrays -ftracer -fforce-addr -freorder-blocks-and-partition
-funroll-loops   -fvisibility-inlines-hidden  -fno-exceptions -fno-check-new
-fno-common   -c /tmp/j1.cc

decoderpluginhandler.cpp: In member function 'aKode::FrameDecoder*
aKode::DecoderPluginHandler::_ZThn24_N5aKode20DecoderPluginHandler16openFrameDecoderEPNS_4FileE(aKode::File*)':
decoderpluginhandler.cpp:65: internal compiler error: Segmentation fault

-- 
           Summary: -freorder-blocks-and-partition causes ICE on 4.1.0
                    (20050604)
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: zlynx at acm dot org
                CC: gcc-bugs at gcc dot gnu dot org
  GCC host triplet: x86_64-pc-linux-gnu


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

[Bug c++/22003] -freorder-blocks-and-partition causes ICE on 4.1.0 (20050604)

[zlynx at acm dot org]

------- Additional Comments From zlynx at acm dot org  2005-06-10 16:45 -------
Created an attachment (id=9062)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9062&action=view)
Preprocessed source code to reproduce ICE


-- 


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-10 21:02 -------
Confirmed, reduced testcase:
struct c1
{
    virtual ~c1();
};
class c4;

struct c2
{
    virtual c4* func();
};

struct c3 : c1, c2
{
    c4* func();
};

c4* c3::func()
{
}


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ctice at apple dot com
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |ice-on-valid-code
   Last reconfirmed|0000-00-00 00:00:00         |2005-06-10 21:02:16
               date|                            |
            Summary|-freorder-blocks-and-       |[4.1 Regression] -freorder-
                   |partition causes ICE on     |blocks-and-partition and
                   |4.1.0 (20050604)            |thunks
   Target Milestone|---                         |4.1.0


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-28 23:16 -------
Patch posted here: <http://gcc.gnu.org/ml/gcc-patches/2005-06/msg02135.html>.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2005-
                   |                            |06/msg02135.html
           Keywords|                            |patch


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2005-07-27 00:01 -------
The problem with the missing CFG is in method.c:use_thunk() 
 
We go through the RTL path without lowering the thunk: 
=============================================== 
  if (this_adjusting 
      && targetm.asm_out.can_output_mi_thunk (thunk_fndecl, fixed_offset, 
                                              virtual_value, alias)) 
    { 
      const char *fnname; 
      current_function_decl = thunk_fndecl; 
      DECL_RESULT (thunk_fndecl) 
        = build_decl (RESULT_DECL, 0, integer_type_node); 
      fnname = XSTR (XEXP (DECL_RTL (thunk_fndecl), 0), 0); 
      init_function_start (thunk_fndecl); 
      current_function_is_thunk = 1; 
      assemble_start_function (thunk_fndecl, fnname); 
 
      targetm.asm_out.output_mi_thunk (asm_out_file, thunk_fndecl, 
                                       fixed_offset, virtual_value, alias); 
 
      assemble_end_function (thunk_fndecl, fnname); 
      current_function_decl = 0; 
      cfun = 0; 
      TREE_ASM_WRITTEN (thunk_fndecl) = 1; 
    } 
=============================================== 
 
Honza, your thoughts please. 
 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu dot
                   |                            |org
   Last reconfirmed|2005-06-10 21:02:16         |2005-07-27 00:01:09
               date|                            |


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2005-07-27 00:46 -------
See http://gcc.gnu.org/ml/gcc-patches/2005-07/msg01755.html for a "similar 
but different" fix. 

-- 


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-27 21:03 -------
Subject: Bug 22003

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	steven@gcc.gnu.org	2005-07-27 21:03:01

Modified files:
	gcc            : ChangeLog varasm.c 

Log message:
	PR c++/22003
	* varasm.c (assemble_start_function): Don't do anything that may
	require a CFG if the current function is a thunk.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.9569&r2=2.9570
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/varasm.c.diff?cvsroot=gcc&r1=1.522&r2=1.523



-- 


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2005-07-27 21:05 -------
This should now be fixed.  Could the reporter check if things work for you 
now?  
 
There may still be the sections issue with -ffunction-sections mentioned in 
http://gcc.gnu.org/ml/gcc-patches/2005-07/msg01829.html.  I'm not sure if 
I should just close this bug and open a new one (not having a test case, etc.) 
or just ignore the issue, or leave this bug open until it is clear what the 
issue really is and what we can do about it... 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2005-07-28 11:17 -------
Accepting this bug so I don't forget about it. 
 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |steven at gcc dot gnu dot
                   |dot org                     |org
             Status|WAITING                     |ASSIGNED
   Last reconfirmed|2005-07-27 00:01:09         |2005-07-28 11:17:43
               date|                            |


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-07-29 12:20 -------
My reduced testcase no longer ICEs.

-- 


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-31 21:45 -------
Subject: Bug 22003

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	steven@gcc.gnu.org	2005-07-31 21:45:44

Modified files:
	gcc            : ChangeLog common.opt postreload-gcse.c 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/g++.dg/other: pr22003.C 
	gcc/testsuite/gfortran.dg: pr23095.f 

Log message:
	PR target/23095
	* common.opt (flag_gcse_after_reload): Don't initialize to 2.
	(flag_rerun_cse_after_loop): Initialize this to 2 instead.
	* postreload-gcse.c (hash_scan_set): Do not consider stack regs.
	
	testsuite/
	PR target/23095
	* gfortran.dg/pr23095.f: New test.
	
	PR c++/22003
	* g++.dg/other/pr22003.C: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.9618&r2=2.9619
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/common.opt.diff?cvsroot=gcc&r1=1.84&r2=1.85
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/postreload-gcse.c.diff?cvsroot=gcc&r1=2.19&r2=2.20
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.5860&r2=1.5861
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/other/pr22003.C.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gfortran.dg/pr23095.f.diff?cvsroot=gcc&r1=NONE&r2=1.1



-- 


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

[Bug c++/22003] [4.1 Regression] -freorder-blocks-and-partition and thunks

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2005-08-01 00:51 -------
It was only a sunny smile, 
and little it cost in the giving, 
but like morning light, 
it scattered the night, 
and made the day 
worth living! 
 

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


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