[Bug c++/58389] New: g++ ICE in ipa_find_reference

[Bug c++/58389] New: g++ ICE in ipa_find_reference

[marc.mutz at kdab dot com]

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

            Bug ID: 58389
           Summary: g++ ICE in ipa_find_reference
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: major
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: marc.mutz at kdab dot com

Created attachment 30796
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30796&action=edit
Preprocessed source (compressed to fit into file size limit)

Compiling master Qt with the trunk version of g++:

Command:
g++ -std=c++11 -o out.o -c -include .pch/release-shared/Qt5Widgets -pipe
-pthread -I/usr/include/gtk-2.0 -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include
-I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/gdk-pixbuf-2.0
-I/usr/include/pango-1.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/glib-2.0
-I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/pixman-1
-I/usr/include/freetype2 -I/usr/include/libpng12 -O2 -fvisibility=hidden
-fvisibility-inlines-hidden -std=c++0x -fno-exceptions -Wall -W -D_REENTRANT
-fPIC -DQT_NO_XKB -DQT_NO_USING_NAMESPACE -DQT_BUILD_WIDGETS_LIB
-DQT_BUILDING_QT -DQT_NO_CAST_TO_ASCII -DQT_ASCII_CAST_WARNINGS -DQT_MOC_COMPAT
-DQT_USE_QSTRINGBUILDER -DQT_DEPRECATED_WARNINGS
-DQT_DISABLE_DEPRECATED_BEFORE=0x050000 -DQT_NO_STYLE_MAC
-DQT_NO_STYLE_WINDOWSVISTA -DQT_NO_STYLE_WINDOWSXP -DQT_NO_STYLE_WINDOWSCE
-DQT_NO_STYLE_WINDOWSMOBILE -DQT_NO_STYLE_ANDROID -DQT_NO_EXCEPTIONS
-D_LARGEFILE64_SOURCE -D_LARGEFILE_SOURCE -DQT_NO_DEBUG -DQT_GUI_LIB
-DQT_CORE_LIB -I/home/marc/qtbase/mkspecs/linux-g++
-I/home/marc/qtbase/src/widgets -I../../include -I../../include/QtWidgets
-I../../include/QtWidgets/5.2.0 -I../../include/QtWidgets/5.2.0/QtWidgets
-I/home/marc/qtbase/src/widgets/dialogs -I.uic/release-shared
-I../../include/QtGui -I../../include/QtGui/5.2.0
-I../../include/QtGui/5.2.0/QtGui -I../../include/QtCore
-I../../include/QtCore/5.2.0 -I../../include/QtCore/5.2.0/QtCore
-I.moc/release-shared -I.
/home/marc/qtbase/src/widgets/graphicsview/qgraphicsitem.cpp -Wall -Wextra
-fno-strict-aliasing -fwrapv -fno-aggressive-loop-optimizations
In file included from
../../include/QtWidgets/5.2.0/QtWidgets/private/qpixmapfilter_p.h:1:0,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/../../../../../../qtbase/src/widgets/effects/qgraphicseffect_p.h:61,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/qgraphicseffect_p.h:1,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/../../../../../../qtbase/src/widgets/kernel/qwidget_p.h:66,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/qwidget_p.h:1,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/../../../../../../qtbase/src/widgets/widgets/qframe_p.h:56,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/qframe_p.h:1,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/../../../../../../qtbase/src/widgets/widgets/qabstractscrollarea_p.h:56,
                 from
../../include/QtWidgets/5.2.0/QtWidgets/private/qabstractscrollarea_p.h:1,
                 from
/home/marc/qtbase/src/widgets/graphicsview/qgraphicsview_p.h:64,
                 from
/home/marc/qtbase/src/widgets/graphicsview/qgraphicsscene_p.h:62,
                 from
/home/marc/qtbase/src/widgets/graphicsview/qgraphicsitem.cpp:725:
../../include/QtWidgets/5.2.0/QtWidgets/private/../../../../../../qtbase/src/widgets/effects/qpixmapfilter_p.h:146:24:
internal compiler error: Segmentation fault
 class Q_WIDGETS_EXPORT QPixmapColorizeFilter : public QPixmapFilter
                        ^
0xacbd0f crash_signal
        ../../gcc/gcc/toplev.c:335
0x98af50 ipa_find_reference(symtab_node_def*, symtab_node_def*,
gimple_statement_d*, unsigned int)
        ../../gcc/gcc/ipa-ref.c:277
0x97ccde remove_described_reference
        ../../gcc/gcc/ipa-prop.c:2510
0x981064 ipa_edge_removal_hook
        ../../gcc/gcc/ipa-prop.c:3022
0x7d9068 cgraph_call_edge_removal_hooks
        ../../gcc/gcc/cgraph.c:314
0x7d9068 cgraph_node_remove_callees(cgraph_node*)
        ../../gcc/gcc/cgraph.c:1563
0x7d9524 cgraph_remove_node(cgraph_node*)
        ../../gcc/gcc/cgraph.c:1680
0x98e154 symtab_remove_unreachable_nodes(bool, _IO_FILE*)
        ../../gcc/gcc/ipa.c:453
0xf2bdb0 ipa_inline
        ../../gcc/gcc/ipa-inline.c:2014
0xf2bdb0 execute
        ../../gcc/gcc/ipa-inline.c:2380
Please submit a full bug report,
with preprocessed source if appropriate.

GCC version:
$ g++ -v
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/opt/gcc/4.9-trunk/libexec/gcc/x86_64-unknown-linux-gnu/4.9.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: ../gcc/configure --prefix=/opt/gcc/4.9-trunk
--enable-languages=c++
Thread model: posix
gcc version 4.9.0 20130911 (experimental) (GCC)

[Bug tree-optimization/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[mpolacek at gcc dot gnu.org]

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

Marek Polacek <mpolacek at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2013-09-11
                 CC|                            |mpolacek at gcc dot gnu.org
          Component|c++                         |tree-optimization
   Target Milestone|---                         |4.9.0
            Summary|g++ ICE in                  |[4.9 Regression] g++ ICE in
                   |ipa_find_reference          |ipa_find_reference
     Ever confirmed|0                           |1

--- Comment #1 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Confirmed.

[Bug tree-optimization/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[markus at trippelsdorf dot de]

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

Markus Trippelsdorf <markus at trippelsdorf dot de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |markus at trippelsdorf dot de

--- Comment #2 from Markus Trippelsdorf <markus at trippelsdorf dot de> ---
Reduced:

template <typename _RandomAccessIterator, typename _Compare>
void __insertion_sort(_RandomAccessIterator, _Compare);
template <typename _RandomAccessIterator, typename _Compare>
void __final_insertion_sort(_RandomAccessIterator p1, _Compare p2) {
  _RandomAccessIterator a;
  if (p1 - a)
    ;
  else
  std:
  __insertion_sort(0, p2);
}
template <typename _RandomAccessIterator, typename _Size, typename _Compare>
void __introsort_loop(_RandomAccessIterator, _Size, _Compare);
template <typename _RandomAccessIterator, typename _Compare>
void sort(_RandomAccessIterator, _RandomAccessIterator p2, _Compare p3) {
std:
  __introsort_loop(0, 0, p3);
  __final_insertion_sort(p2, p3);
}
class A {
public:
  int m_fn1();
  void __lg();
  class B {
  public:
    int i;
    int operator-(B);
  };
};
class C;
class D {
public:
  C *operator->();
};
class F {
  A m_fn1() const;
  D d_ptr;
};
class C {
  friend F;
  void m_fn1();
  A children;
};
void qt_notclosestLeaf();
inline void C::m_fn1() {
  A::B b, c;
  if (children.m_fn1()) {
    sort(c, b, qt_notclosestLeaf);
  }
}
A F::m_fn1() const { const_cast<F *>(this)->d_ptr->m_fn1(); }

[Bug tree-optimization/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[mpolacek at gcc dot gnu.org]

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

--- Comment #4 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Started with r202281.

[Bug tree-optimization/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jamborm at gcc dot gnu.org

--- Comment #3 from Martin Jambor <jamborm at gcc dot gnu.org> ---
This also seems like I should have a look.

[Bug ipa/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
          Component|tree-optimization           |ipa
           Assignee|unassigned at gcc dot gnu.org      |jamborm at gcc dot gnu.org

--- Comment #5 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Mine.  I'm testing a fix.

[Bug ipa/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-p
                   |                            |atches/2013-09/msg00914.htm
                   |                            |l

--- Comment #6 from Martin Jambor <jamborm at gcc dot gnu.org> ---
I have posted the patch to the mailing list:

http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00914.html

[Bug ipa/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[jamborm at gcc dot gnu.org]

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

--- Comment #7 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Thu Sep 12 15:20:05 2013
New Revision: 202532

URL: http://gcc.gnu.org/viewcvs?rev=202532&root=gcc&view=rev
Log:
2013-09-12  Martin Jambor  <mjambor@suse.cz>

    PR ipa/58389
    * ipa-prop.c (remove_described_reference): Give up if the edge in the
    reference descriptor is NULL.
    (ipa_edge_removal_hook): If owning a reference descriptor, set its
    edge to NULL.

testsuite/
    * g++.dg/pr58389.C: New test.


Added:
    trunk/gcc/testsuite/g++.dg/pr58389.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-prop.c
    trunk/gcc/testsuite/ChangeLog

[Bug ipa/58389] [4.9 Regression] g++ ICE in ipa_find_reference

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #8 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Fixed.