[Bug c++/28116] New: ICE when building konverter with gcc-4.1 with -O3

[Bug c++/28116] New: ICE when building konverter with gcc-4.1 with -O3

[vapier at gentoo dot org]

when building konverter with gcc-4.1.1, gcc ICEs:

kxinewidget.cpp: In member function 'QTime KXineWidget::getLengthInfo()':
kxinewidget.cpp:2841: internal compiler error: Segmentation fault
Please submit a full bug report

gcc-4.0.3 and gcc-4.2 compiles the code just fine

validated on Gentoo and Debian


-- 
           Summary: ICE when building konverter with gcc-4.1 with -O3
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: vapier at gentoo dot org
  GCC host triplet: i686-linux-gnu
GCC target triplet: i686-linux-gnu


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

[Bug c++/28116] ICE when building konverter with gcc-4.1 with -O3

[vapier at gentoo dot org]

------- Comment #1 from vapier at gentoo dot org  2006-06-21 04:04 -------
Created an attachment (id=11716)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11716&action=view)
PR28116-ice.ii


-- 


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

[Bug c++/28116] ICE when building konverter with gcc-4.1 with -O3

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2006-06-21 04:08 -------
Works just fine with "4.1.0 20060208" but fails with "4.1.2 20060608" and only
with -m32 -O3.
pr28116.ii:17: internal compiler error: in expand_expr_addr_expr_1, at
expr.c:6336
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.


-- 


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

[Bug c++/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-06-21 04:14 -------
*** Bug 28117 has been marked as a duplicate of this bug. ***


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |middle-end
            Summary|[4.1 Regression] ICE when   |[4.1 Regression] ICE when
                   |building konverter with gcc-|building konverter with gcc-
                   |4.1 with -O3                |4.1 with -O3 [RSO]
   Target Milestone|---                         |4.1.2


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[jakub at gcc dot gnu dot org]

------- Comment #4 from jakub at gcc dot gnu dot org  2006-06-21 12:20 -------
This sounds like a tree-nrv bug to me.
We have at *.optimized:
QTime QTime::msToTime() (this)
{
  struct QTimeD.1732 D.1772;
  struct QTimeD.1732 tD.1765;

  (voidD.34) 0;
  D.1772 = addMSecs (&<retval>);
  <retval> = D.1772;
  return <retval>;
}

(note D.1772 never has address taken, while <retval> does).
Now tree_nrv () finds <retval> is only set to D.1772 and:
  /* At this point we know that all the return statements return the
     same local which has suitable attributes for NRV.   Copy debugging
     information from FOUND to RESULT.  */
  DECL_NAME (result) = DECL_NAME (found);
  DECL_SOURCE_LOCATION (result) = DECL_SOURCE_LOCATION (found);
  DECL_ABSTRACT_ORIGIN (result) = DECL_ABSTRACT_ORIGIN (found);
  TREE_ADDRESSABLE (result) = TREE_ADDRESSABLE (found);
which clears the addressable flag, although the RESULT_DECL clearly has its
address taken.  Perhaps the last line should be:
TREE_ADDRESSABLE (result) |= TREE_ADDRESSABLE (found);
but am not 100% sure if tree_nrv is a safe transformation if TREE_ADDRESSABLE
(result) was set already.  Also, in this case DECL_NAME (result) contained the
real var name (t) while DECL_NAME (found) is NULL, so perhaps the first 3 lines
should be conditionalized on if (DECL_NAME (found)) to avoid making debug info
worse.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dnovillo at redhat dot com,
                   |                            |law at gcc dot gnu dot org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-06-21 12:20:43
               date|                            |


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[jakub at gcc dot gnu dot org]

------- Comment #5 from jakub at gcc dot gnu dot org  2006-06-22 16:58 -------
In GCC trunk bootstrap + regression testing the tree-nrv optimization hits 584
types, out of this 38 cases have TREE_ADDRESSABLE set on the RESULT_DECL
(because C++ FE already NRV optimized it, address of <retval> was taken usually
when it was constructed, then some optimizations later on set <retval> to
a temporary variable and the code that takes &<result> was optimized out), e.g.
when compiling fstream-inst.cc in libstdc++-v3.
So, bailing out in tree_nrv whenever TREE_ADDRESSABLE (result) would pessimize
those.  On the other side, in tree_nrv we know that TREE_ADDRESSABLE (found) ==
0 (we check for that), so we can either stop copying the addressable flag, or
bail if TREE_ADDRESSABLE (result).


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2006-08-25 06:05 -------
*** Bug 28840 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |eemax at web dot de


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[bonzini at gnu dot org]

------- Comment #7 from bonzini at gnu dot org  2006-08-25 06:33 -------
reduced testcase from pr28840

struct QDateTime
{
    QDateTime addSecs( int secs ) const;
    int t;
};
QDateTime gridToDate(long x)
{
  QDateTime date;
  date = date.addSecs(1);
  return date;
}
void whatsOnAt(long x, long y)
{
  gridToDate(x);
}


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #8 from rguenth at gcc dot gnu dot org  2006-10-18 09:46 -------
Janis, can you check what fixed this on the mainline?  Thanks!


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |janis at gcc dot gnu dot org
      Known to fail|4.1.1                       |4.1.0 4.1.1
   Last reconfirmed|2006-06-21 12:20:43         |2006-10-18 09:46:51
               date|                            |


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[janis at gcc dot gnu dot org]

------- Comment #9 from janis at gcc dot gnu dot org  2006-10-19 16:35 -------
I wasn't able to do a regression hunt for the fix on the mainline because I
didn't find any revision for which the test failed on the mainline.

A regression hunt on the 4.1 branch using the testcase from comment #7 with an
i686-linux cross compiler identified this patch where the test starts getting
the ICE:

    http://gcc.gnu.org/viewcvs?view=rev&rev=110838

    r110838 | jason | 2006-02-10 17:32:10 +0000 (Fri, 10 Feb 2006)

The mainline didn't ICE when that patch was added.


-- 

janis at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu dot org


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[pinskia at gcc dot gnu dot org]

------- Comment #10 from pinskia at gcc dot gnu dot org  2006-11-12 17:07 -------
*** Bug 29809 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |chris_clayton at f1internet
                   |                            |dot com


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[chris_clayton at f1internet dot com]

------- Comment #11 from chris_clayton at f1internet dot com  2006-11-26 12:54 -------
I'm the reporter of 29809, which is a duplicate of this bug. I'm not familiar
with the workflow here, so do I simply wait for 4.1.2 or is there likely to be
patch to test before 4.1.2 is released? Is the patch at comment 4 of bug 29809
worth testing - the author seemed to think it was too simple to be true :)?

Thanks


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #12 from rguenth at gcc dot gnu dot org  2006-11-26 13:09 -------
Usually you wait until a patch is available or 4.1.2 is released (supposed the
problem is fixed there).  You can try testing the ad-hoc fix in PR29809 comment
#4.


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[chris_clayton at f1internet dot com]

------- Comment #13 from chris_clayton at f1internet dot com  2006-11-26 21:22 -------
(In reply to comment #12)
> Usually you wait until a patch is available or 4.1.2 is released (supposed the
> problem is fixed there).  You can try testing the ad-hoc fix in PR29809 comment
> #4.
> 

OK, I've updated to the 20061124 snapshot. Without the patch at PR29809, I get
the ICE building avifile-0.7.45 and konverter-0.92-beta1. With the patch, both
builds complete. From my, admittedly minimal, testing, konverter seems to be
working OK. Unfortunately, I don't have the hardware to test the sample
application from the avifile package.

Hope this helps.


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #14 from rguenth at gcc dot gnu dot org  2007-01-05 11:43 -------
Mine.  I'm testing the inliner fix.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-10-18 09:46:51         |2007-01-05 11:43:22
               date|                            |


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #15 from rguenth at gcc dot gnu dot org  2007-01-05 13:53 -------
Subject: Bug 28116

Author: rguenth
Date: Fri Jan  5 13:53:45 2007
New Revision: 120480

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120480
Log:
2007-01-05  Richard Guenther  <rguenther@suse.de>

        PR middle-end/28116
        * tree-inline.c (copy_body_r): Mark argument of ADDR_EXPR
        addressable.

        * g++.dg/opt/pr28116.C: New testcase.

Added:
    branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C
Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_1-branch/gcc/tree-inline.c


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #16 from rguenth at gcc dot gnu dot org  2007-01-05 13:57 -------
Subject: Bug 28116

Author: rguenth
Date: Fri Jan  5 13:56:54 2007
New Revision: 120481

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120481
Log:
2007-01-05  Richard Guenther  <rguenther@suse.de>

        PR middle-end/28116
        * g++.dg/opt/pr28116.C: New testcase.

Added:
    branches/gcc-4_2-branch/gcc/testsuite/g++.dg/opt/pr28116.C
      - copied unchanged from r120480,
branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C
Modified:
    branches/gcc-4_2-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #17 from rguenth at gcc dot gnu dot org  2007-01-05 14:00 -------
Subject: Bug 28116

Author: rguenth
Date: Fri Jan  5 14:00:46 2007
New Revision: 120482

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=120482
Log:
2007-01-05  Richard Guenther  <rguenther@suse.de>

        PR middle-end/28116
        * g++.dg/opt/pr28116.C: New testcase.

Added:
    trunk/gcc/testsuite/g++.dg/opt/pr28116.C
      - copied unchanged from r120480,
branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C
Modified:
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[rguenth at gcc dot gnu dot org]

------- Comment #18 from rguenth at gcc dot gnu dot org  2007-01-05 14:02 -------
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

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


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[tbm at cyrius dot com]

------- Comment #19 from tbm at cyrius dot com  2007-02-03 09:47 -------
(In reply to comment #18)
> Fixed.

Richi, do you think you can check whether PR28358 is really a duplicate of this 
bug (as Andrew thinks) and should be closed.


-- 


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

[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]

[pinskia at gcc dot gnu dot org]

------- Comment #20 from pinskia at gcc dot gnu dot org  2007-08-24 00:24 -------
*** Bug 28358 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |peter dot soetens at fmtc
                   |                            |dot be


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