[Bug c++/44039] New: [4.5 regression] ICE: Segmentation fault on error recovery

[Bug c++/44039] New: [4.5 regression] ICE: Segmentation fault on error recovery

[doko at ubuntu dot com]

$ g++ -c logTargets.ii 
In file included from
/usr/include/boost/date_time/gregorian/formatters.hpp:17:0,
                 from /usr/include/boost/date_time/gregorian/gregorian.hpp:25,
                 from
/usr/include/boost/date_time/posix_time/time_formatters.hpp:12,
                 from
/usr/include/boost/date_time/posix_time/posix_time.hpp:24,
                 from logTargets.cpp:48:
/usr/include/boost/date_time/date_formatting.hpp: In static member function
'static std::basic_string<charT> boost::date_time::ymd_formatter<ymd_type,
format_type, charT>::ymd_to_string(ymd_type)':
/usr/include/boost/date_time/date_formatting.hpp:87:36: error: cannot call
constructor 'std::locale::locale' directly
/usr/include/boost/date_time/date_formatting.hpp:87:36: error:   for a
function-style cast, remove the redundant '::locale'
/usr/include/boost/date_time/date_formatting.hpp: In static member function
'static std::basic_string<charT> boost::date_time::ymd_formatter<ymd_type,
format_type, charT>::ymd_to_string(ymd_type) [with ymd_type =
boost::date_time::year_month_day_base<boost::gregorian::greg_year,
boost::gregorian::greg_month, boost::gregorian::greg_day>, format_type =
boost::date_time::simple_format<char>, charT = char]':
/usr/include/boost/date_time/date_formatting.hpp:124:76:   instantiated from
'static boost::date_time::date_formatter<date_type, format_type,
charT>::string_type boost::date_time::date_formatter<date_type, format_type,
charT>::date_to_string(date_type) [with date_type = boost::gregorian::date,
format_type = boost::date_time::simple_format<char>, charT = char,
boost::date_time::date_formatter<date_type, format_type, charT>::string_type =
std::basic_string<char>]'
/usr/include/boost/date_time/gregorian/formatters.hpp:34:99:   instantiated
from 'std::basic_string<_CharT> boost::gregorian::to_simple_string_type(const
boost::gregorian::date&) [with charT = char]'
/usr/include/boost/date_time/gregorian/formatters.hpp:40:41:   instantiated
from here
/usr/include/boost/date_time/date_formatting.hpp:87:7: internal compiler error:
Segmentation fault
Please submit a full bug report,
with preprocessed source if appropriate.


-- 
           Summary: [4.5 regression] ICE: Segmentation fault on error
                    recovery
           Product: gcc
           Version: 4.5.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: doko at ubuntu dot com


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

[Bug c++/44039] [4.5 regression] ICE: Segmentation fault on error recovery

[doko at ubuntu dot com]

------- Comment #1 from doko at ubuntu dot com  2010-05-08 14:07 -------
Created an attachment (id=20602)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20602&action=view)
preprocessed source


-- 


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

[Bug c++/44039] [4.5 regression] ICE: Segmentation fault on error recovery

[doko at ubuntu dot com]

------- Comment #2 from doko at ubuntu dot com  2010-05-08 14:08 -------
seen with 4.5 20100506 on x86_64-linux-gnu


-- 

doko at ubuntu dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |4.5.0
      Known to work|                            |4.4.4


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

[Bug c++/44039] [4.5 regression] ICE: Segmentation fault on error recovery

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |error-recovery
   Target Milestone|---                         |4.5.1


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

[Bug c++/44039] [4.5 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #3 from paolo dot carlini at oracle dot com  2010-05-08 18:28 -------
Matthias, can you please reduce to a manageable size these testcases? Thanks in
advance.


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2010-05-19 12:56 -------
Confirmed.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |ice-on-invalid-code
           Priority|P3                          |P5
   Last reconfirmed|0000-00-00 00:00:00         |2010-05-19 12:56:51
               date|                            |
            Summary|[4.5 regression] ICE:       |[4.5/4.6 regression] ICE:
                   |Segmentation fault on error |Segmentation fault on error
                   |recovery                    |recovery


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #5 from paolo dot carlini at oracle dot com  2010-06-27 08:59 -------
Reducing...


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #6 from paolo dot carlini at oracle dot com  2010-06-28 13:08 -------
I'm still working on this, but this kind of situation is certainly part of the
problem, if not all of it, it ICES.

Jason, does it look familiar to you? I'm asking because I think you implemented
the recent changes to reject locale::locale() below...

////////////////

struct locale {  };

template<class charT>
  void
  foo()
  { locale::locale(); }

void
bar()
{ foo<char>(); }


-- 

paolo dot carlini at oracle dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |paolo dot carlini at oracle
                   |                            |dot com, jason at gcc dot
                   |                            |gnu dot org


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #7 from paolo dot carlini at oracle dot com  2010-06-28 15:20 -------
I'm now pretty sure that the issue with locale::locale() is all there is to
this PR.  I also checked that the below patchlet avoids the ICE both for the
original testcase and my snippet (it also passes the testsuite, but I'm not
sure whether we could catch the problem earlier):

Index: pt.c
===================================================================
--- pt.c        (revision 161491)
+++ pt.c        (working copy)
@@ -10695,6 +10695,8 @@ tsubst_baselink (tree baselink, tree object_type,
     if (IDENTIFIER_TYPENAME_P (name))
       name = mangle_conv_op_name_for_type (optype);
     baselink = lookup_fnfields (qualifying_scope, name, /*protect=*/1);
+    if (!baselink)
+      return error_mark_node;

     /* If lookup found a single function, mark it as used at this
        point.  (If it lookup found multiple functions the one selected


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[jason at gcc dot gnu dot org]

------- Comment #8 from jason at gcc dot gnu dot org  2010-06-30 18:27 -------
That patch is OK, it's reasonable for lookup_fnfields to return NULL.


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[jason at gcc dot gnu dot org]

------- Comment #9 from jason at gcc dot gnu dot org  2010-06-30 18:28 -------
*** Bug 44040 has been marked as a duplicate of this bug. ***


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #10 from paolo dot carlini at oracle dot com  2010-06-30 18:43 -------
Thanks Jason, I'll add the testcase, regtest again and post before committing.


-- 

paolo dot carlini at oracle dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |paolo dot carlini at oracle
                   |dot org                     |dot com
             Status|NEW                         |ASSIGNED


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo at gcc dot gnu dot org]

------- Comment #11 from paolo at gcc dot gnu dot org  2010-07-02 10:13 -------
Subject: Bug 44039

Author: paolo
Date: Fri Jul  2 10:13:21 2010
New Revision: 161698

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161698
Log:
/cp
2010-07-02  Paolo Carlini  <paolo.carlini@oracle.com>

        PR c++/44039
        * pt.c (tsubst_baselink): Return error_mark_node if lookup_fnfields
        returns NULL_TREE.

/testsuite
2010-07-02  Paolo Carlini  <paolo.carlini@oracle.com>

        PR c++/44039
        * g++.dg/template/crash101.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/template/crash101.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/pt.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo at gcc dot gnu dot org]

------- Comment #12 from paolo at gcc dot gnu dot org  2010-07-02 10:15 -------
Subject: Bug 44039

Author: paolo
Date: Fri Jul  2 10:15:10 2010
New Revision: 161700

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161700
Log:
/cp
2010-07-02  Paolo Carlini  <paolo.carlini@oracle.com>

        PR c++/44039
        * pt.c (tsubst_baselink): Return error_mark_node if lookup_fnfields
        returns NULL_TREE.

/testsuite
2010-07-02  Paolo Carlini  <paolo.carlini@oracle.com>

        PR c++/44039
        * g++.dg/template/crash98.C: New.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/g++.dg/template/crash98.C
Modified:
    branches/gcc-4_5-branch/gcc/cp/ChangeLog
    branches/gcc-4_5-branch/gcc/cp/pt.c
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/44039] [4.5/4.6 regression] ICE: Segmentation fault on error recovery

[paolo dot carlini at oracle dot com]

------- Comment #13 from paolo dot carlini at oracle dot com  2010-07-02 10:17 -------
Fixed for 4.5.1.


-- 

paolo dot carlini at oracle dot com changed:

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


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