[Bug debug/48703] New: segfault in canonicalize_for_substitution

[Bug debug/48703] New: segfault in canonicalize_for_substitution

[matz at gcc dot gnu.org]

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

           Summary: segfault in canonicalize_for_substitution
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: debug
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: matz@gcc.gnu.org


Created attachment 24062
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=24062
testcase

The patch for PR48207 broke compiling any moderately complex C++ program with
-g -flto.  We can't use the C++ specific langhook for mangling when we've
already cleared out language specific stuff in free_lang_data.

$ cc1plus -g -flto bug.ii
/matz/gcc/svn/real-trunk/dev/x86_64-unknown-linux-gnu/libstdc++-v3/include/istream:
In instantiation of ‘std::basic_istream<_CharT, _Traits>::sentry::operator
bool() const [with _CharT = wchar_t, _Traits = std::char_traits<wchar_t>]’:
EtherAppCli.cc:221:1:   instantiated from here
/matz/gcc/svn/real-trunk/dev/x86_64-unknown-linux-gnu/libstdc++-v3/include/istream:686:7:
internal compiler error: Segmentation fault

[Bug debug/48703] segfault in canonicalize_for_substitution

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011.04.21 12:19:17
             Blocks|                            |48207
         AssignedTo|unassigned at gcc dot       |rguenth at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-04-21 12:19:17 UTC ---
I have a patch.

[Bug debug/48703] segfault in canonicalize_for_substitution

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

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

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-04-21 14:35:56 UTC ---
Fixed.

[Bug debug/48703] segfault in canonicalize_for_substitution

[rguenth at gcc dot gnu.org]

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

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-04-21 14:35:46 UTC ---
Author: rguenth
Date: Thu Apr 21 14:35:43 2011
New Revision: 172830

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=172830
Log:
2011-04-21  Richard Guenther  <rguenther@suse.de>

    PR lto/48703
    * tree.c (free_lang_data_in_decl): Do not zero TREE_TYPE of
    DECL_NAME.

    * g++.dg/lto/pr48207-2_0.C: New testcase.
    * g++.dg/lto/pr48207-3_0.C: Likewise.

Added:
    trunk/gcc/testsuite/g++.dg/lto/pr48207-2_0.C
    trunk/gcc/testsuite/g++.dg/lto/pr48207-3_0.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree.c

[Bug debug/48703] segfault in canonicalize_for_substitution

[rguenth at gcc dot gnu.org]

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

--- Comment #4 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-05-18 13:29:28 UTC ---
Author: rguenth
Date: Wed May 18 13:29:24 2011
New Revision: 173860

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

        Backport from mainline
        2011-04-19  Richard Guenther  <rguenther@suse.de>

    PR lto/48207
    * tree.c (free_lang_data): Do not reset the decl-assembler-name
    langhook.

    * g++.dg/lto/pr48207_0.C: New testcase.

        2011-04-21  Richard Guenther  <rguenther@suse.de>

    PR lto/48703
    * tree.c (free_lang_data_in_decl): Do not zero TREE_TYPE of
    DECL_NAME.

    * g++.dg/lto/pr48207-2_0.C: New testcase.
    * g++.dg/lto/pr48207-3_0.C: Likewise.

Added:
    branches/gcc-4_6-branch/gcc/testsuite/g++.dg/lto/pr48207-2_0.C
    branches/gcc-4_6-branch/gcc/testsuite/g++.dg/lto/pr48207-3_0.C
    branches/gcc-4_6-branch/gcc/testsuite/g++.dg/lto/pr48207_0.C
Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_6-branch/gcc/tree.c