[Bug c++/41275] [4.5] ICE: expand_expr_real_1, at expr.c:8416

[Bug c++/41275] [4.5] ICE: expand_expr_real_1, at expr.c:8416

[rubidium at openttd dot org]

------- Comment #1 from rubidium at openttd dot org  2009-09-05 13:42 -------
Created an attachment (id=18506)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18506&action=view)
The .ii file of save-temps


-- 


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

[Bug c++/41275] New: [4.5] ICE: expand_expr_real_1, at expr.c:8416

[rubidium at openttd dot org]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 7326 bytes --]

With GCC trunk@151445

g++-4.5 -v -save-temps -O2 -fomit-frame-pointer  -DUNIX -Wall -Wno-multichar
-Wsign-compare -Wundef -Wwrite-strings -Wpointer-arith -Wno-uninitialized -W
-Wno-unused-parameter -Wformat=2 -Wredundant-decls -fno-strict-aliasing
-Wcast-qual -fno-strict-overflow -DWITH_ALLEGRO -I/usr/include -DWITH_SDL
-I/usr/include/SDL  -D_REENTRANT -DWITH_ZLIB -D_SQ64
-I/home/rubidium/openttd/special/gcc-4.5/src/3rdparty/squirrel/include
-DWITH_PNG  -I/usr/include/libpng12  -DWITH_FONTCONFIG    -DWITH_FREETYPE
-I/usr/include/freetype2  -DWITH_ICU -D_REENTRANT  -I/usr/include  
-DENABLE_NETWORK -DWITH_PERSONAL_DIR -DPERSONAL_DIR=\".openttd\"
-DGLOBAL_DATA_DIR=\"/usr/local/share/games/openttd\" -I
/home/rubidium/openttd/special/gcc-4.5/objs/release -I
/home/rubidium/openttd/special/gcc-4.5/objs/lang -c -o misc_gui.o
/home/rubidium/openttd/special/gcc-4.5/src/misc_gui.cpp                         
Using built-in specs.                                                           
Target: x86_64-unknown-linux-gnu                                                
Configured with: ../4.5/configure --program-suffix=-4.5
--enable-languages=c,c++ : (reconfigured) ../4.5/configure
--program-suffix=-4.5 --enable-languages=c,c++ --no-create --no-recursion :
(reconfigured) ../4.5/configure --program-suffix=-4.5 --enable-languages=c,c++
--no-create --no-recursion : (reconfigured) ../4.5/configure
--program-suffix=-4.5 --enable-languages=c,c++ --no-create --no-recursion       
Thread model: posix                                                             
gcc version 4.5.0 20090905 (experimental) (GCC)                                 
COLLECT_GCC_OPTIONS='-v' '-save-temps' '-O2' '-fomit-frame-pointer' '-DUNIX'
'-Wall' '-Wno-multichar' '-Wsign-compare' '-Wundef' '-Wwrite-strings'
'-Wpointer-arith' '-Wno-uninitialized' '-W' '-Wno-unused-parameter'
'-Wformat=2' '-Wredundant-decls' '-fno-strict-aliasing' '-Wcast-qual'
'-fno-strict-overflow' '-DWITH_ALLEGRO' '-I/usr/include' '-DWITH_SDL'
'-I/usr/include/SDL' '-D_REENTRANT' '-DWITH_ZLIB' '-D_SQ64'
'-I/home/rubidium/openttd/special/gcc-4.5/src/3rdparty/squirrel/include'
'-DWITH_PNG' '-I/usr/include/libpng12' '-DWITH_FONTCONFIG' '-DWITH_FREETYPE'
'-I/usr/include/freetype2' '-DWITH_ICU' '-D_REENTRANT' '-I/usr/include'
'-DENABLE_NETWORK' '-DWITH_PERSONAL_DIR' '-DPERSONAL_DIR=".openttd"'
'-DGLOBAL_DATA_DIR="/usr/local/share/games/openttd"' '-I'
'/home/rubidium/openttd/special/gcc-4.5/objs/release' '-I'
'/home/rubidium/openttd/special/gcc-4.5/objs/lang' '-c' '-o' 'misc_gui.o'
'-shared-libgcc' '-mtune=generic'
 /usr/local/libexec/gcc/x86_64-unknown-linux-gnu/4.5.0/cc1plus -E -quiet -v
-I/usr/include -I/usr/include/SDL
-I/home/rubidium/openttd/special/gcc-4.5/src/3rdparty/squirrel/include
-I/usr/include/libpng12 -I/usr/include/freetype2 -I/usr/include -I
/home/rubidium/openttd/special/gcc-4.5/objs/release -I
/home/rubidium/openttd/special/gcc-4.5/objs/lang -D_GNU_SOURCE -DUNIX
-DWITH_ALLEGRO -DWITH_SDL -D_REENTRANT -DWITH_ZLIB -D_SQ64 -DWITH_PNG
-DWITH_FONTCONFIG -DWITH_FREETYPE -DWITH_ICU -D_REENTRANT -DENABLE_NETWORK
-DWITH_PERSONAL_DIR -DPERSONAL_DIR=".openttd"
-DGLOBAL_DATA_DIR="/usr/local/share/games/openttd"
/home/rubidium/openttd/special/gcc-4.5/src/misc_gui.cpp -mtune=generic -Wall
-Wno-multichar -Wsign-compare -Wundef -Wwrite-strings -Wpointer-arith
-Wno-uninitialized -W -Wno-unused-parameter -Wformat=2 -Wredundant-decls
-Wcast-qual -fomit-frame-pointer -fno-strict-aliasing -fno-strict-overflow -O2
-fpch-preprocess -o misc_gui.ii
ignoring nonexistent directory
"/usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../x86_64-unknown-linux-gnu/include"
ignoring duplicate directory "/usr/include"
  as it is a non-system directory that duplicates a system directory
ignoring duplicate directory "/usr/include"
  as it is a non-system directory that duplicates a system directory
#include "..." search starts here:
#include <...> search starts here:
 /usr/include/SDL
 /home/rubidium/openttd/special/gcc-4.5/src/3rdparty/squirrel/include
 /usr/include/libpng12
 /usr/include/freetype2
 /home/rubidium/openttd/special/gcc-4.5/objs/release
 /home/rubidium/openttd/special/gcc-4.5/objs/lang

/usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../include/c++/4.5.0

/usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../include/c++/4.5.0/x86_64-unknown-linux-gnu

/usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../include/c++/4.5.0/backward
 /usr/local/include
 /usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/include
 /usr/local/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/include-fixed
 /usr/include
End of search list.
COLLECT_GCC_OPTIONS='-v' '-save-temps' '-O2' '-fomit-frame-pointer' '-DUNIX'
'-Wall' '-Wno-multichar' '-Wsign-compare' '-Wundef' '-Wwrite-strings'
'-Wpointer-arith' '-Wno-uninitialized' '-W' '-Wno-unused-parameter'
'-Wformat=2' '-Wredundant-decls' '-fno-strict-aliasing' '-Wcast-qual'
'-fno-strict-overflow' '-DWITH_ALLEGRO' '-I/usr/include' '-DWITH_SDL'
'-I/usr/include/SDL' '-D_REENTRANT' '-DWITH_ZLIB' '-D_SQ64'
'-I/home/rubidium/openttd/special/gcc-4.5/src/3rdparty/squirrel/include'
'-DWITH_PNG' '-I/usr/include/libpng12' '-DWITH_FONTCONFIG' '-DWITH_FREETYPE'
'-I/usr/include/freetype2' '-DWITH_ICU' '-D_REENTRANT' '-I/usr/include'
'-DENABLE_NETWORK' '-DWITH_PERSONAL_DIR' '-DPERSONAL_DIR=".openttd"'
'-DGLOBAL_DATA_DIR="/usr/local/share/games/openttd"' '-I'
'/home/rubidium/openttd/special/gcc-4.5/objs/release' '-I'
'/home/rubidium/openttd/special/gcc-4.5/objs/lang' '-c' '-o' 'misc_gui.o'
'-shared-libgcc' '-mtune=generic'
 /usr/local/libexec/gcc/x86_64-unknown-linux-gnu/4.5.0/cc1plus -fpreprocessed
misc_gui.ii -quiet -dumpbase misc_gui.cpp -mtune=generic -auxbase-strip
misc_gui.o -O2 -Wall -Wno-multichar -Wsign-compare -Wundef -Wwrite-strings
-Wpointer-arith -Wno-uninitialized -W -Wno-unused-parameter -Wformat=2
-Wredundant-decls -Wcast-qual -version -fomit-frame-pointer
-fno-strict-aliasing -fno-strict-overflow -o misc_gui.s
GNU C++ (GCC) version 4.5.0 20090905 (experimental) (x86_64-unknown-linux-gnu)
        compiled by GNU C version 4.5.0 20090904 (experimental), GMP version
4.3.1, MPFR version 2.4.1-p2
GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096
GNU C++ (GCC) version 4.5.0 20090905 (experimental) (x86_64-unknown-linux-gnu)
        compiled by GNU C version 4.5.0 20090904 (experimental), GMP version
4.3.1, MPFR version 2.4.1-p2
GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096
Compiler executable checksum: 7221e3a3fa11525f4d349f716eeef094
/home/rubidium/openttd/special/gcc-4.5/src/misc_gui.cpp: In member function
‘ErrmsgWindow::~ErrmsgWindow()’:
/home/rubidium/openttd/special/gcc-4.5/src/misc_gui.cpp:640:44: internal
compiler error: in expand_expr_real_1, at expr.c:8416


-- 
           Summary: [4.5] ICE: expand_expr_real_1, at expr.c:8416
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rubidium at openttd dot org
 GCC build triplet: x86_64-unknown-linux-gnu
  GCC host triplet: x86_64-unknown-linux-gnu
GCC target triplet: x86_64-unknown-linux-gnu


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

[Bug c++/41275] [4.5] ICE: expand_expr_real_1, at expr.c:8416

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2009-09-05 13:53 -------
Reducing.


-- 


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2009-09-05 14:22 -------
typedef unsigned short StringID;
static const StringID INVALID_STRING_ID = 0xFFFF;
class ZeroedMemoryAllocator {
public:
    virtual ~ZeroedMemoryAllocator() { }
};
struct Window : ZeroedMemoryAllocator {
    Window();
};
struct ErrmsgWindow : public Window 
{
    ErrmsgWindow(bool show_company_manager_face) : Window() { }
    ~ErrmsgWindow()  
    {
      extern StringID _switch_mode_errorstr;
      _switch_mode_errorstr = INVALID_STRING_ID;
    }
};
void ShowErrorMessage(void)
{
  ErrmsgWindow w(false);
}


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rguenth at gcc dot gnu dot
                   |                            |org, matz at suse dot de
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-09-05 14:22:36
               date|                            |
            Summary|[4.5] ICE:                  |[4.5 Regression] ICE:
                   |expand_expr_real_1, at      |expand_expr_real_1, at
                   |expr.c:8416                 |expr.c:8416
   Target Milestone|---                         |4.5.0


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[matz at gcc dot gnu dot org]

------- Comment #4 from matz at gcc dot gnu dot org  2009-09-09 15:41 -------
Even shorter:
struct ErrmsgWindow
{
    virtual ~ErrmsgWindow()
    {
      extern int _switch_mode_errorstr;
      _switch_mode_errorstr = 42;
    }
};
void ShowErrorMessage(void)
{
  ErrmsgWindow w;
}

The problem appears with r151360, which is the merge of free_lang_data.

A quick glance reveals that the problem is that somehow the inlining of
__deleting_dtor into __comp_dtor doesn't correctly adjust the DECL_CONTEXT
of the _switch_mode_errorstr decl.

As this can't be reproduced with a similar setup just involving normal
functions (not dtors) I would assume that the special handling of clones
and how they are produced from each other is broken.


-- 

matz at gcc dot gnu dot org changed:

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


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[matz at gcc dot gnu dot org]

------- Comment #5 from matz at gcc dot gnu dot org  2009-09-09 15:46 -------
Oh, maybe an explanation: what is expected is that such decls (externs) either
have no context set (global) or have the current function as context.  For
inlining (cloning/versioning all the same) this needs to be done by remapping
the decls.  No idea why this isn't called on the _switch_mode_errorstr
in the inliner.  As this works with normal functions I assume that somehow
the free_lang_data work changed internal structures in a way that this decl
isn't visible to the inliner anymore (in a way).


-- 


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[rguenther at suse dot de]

------- Comment #6 from rguenther at suse dot de  2009-09-10 08:42 -------
Subject: Re:  [4.5 Regression] ICE: expand_expr_real_1, at
 expr.c:8416

On Wed, 9 Sep 2009, matz at gcc dot gnu dot org wrote:

> ------- Comment #5 from matz at gcc dot gnu dot org  2009-09-09 15:46 -------
> Oh, maybe an explanation: what is expected is that such decls (externs) either
> have no context set (global) or have the current function as context.  For
> inlining (cloning/versioning all the same) this needs to be done by remapping
> the decls.  No idea why this isn't called on the _switch_mode_errorstr
> in the inliner.  As this works with normal functions I assume that somehow
> the free_lang_data work changed internal structures in a way that this decl
> isn't visible to the inliner anymore (in a way).

It changes most DECL_CONTEXTs to NULL anyway.  Cloning is running way
before free_lang_data, so if it is working without fld then it really
must be a fld issue...

Richard.


-- 


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[matz at gcc dot gnu dot org]

------- Comment #7 from matz at gcc dot gnu dot org  2009-09-11 11:08 -------
Subject: Bug 41275

Author: matz
Date: Fri Sep 11 11:08:38 2009
New Revision: 151631

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151631
Log:
        PR middle-end/41275
        * tree-inline.c (remap_decls): Don't put DECL_EXTERNAL decls
        on the local_decls list.

testsuite/
        * g++.dg/tree-ssa/pr41275.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/tree-ssa/pr41275.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-inline.c


-- 


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[matz at gcc dot gnu dot org]

------- Comment #8 from matz at gcc dot gnu dot org  2009-09-11 12:13 -------
Fixed.


-- 

matz at gcc dot gnu dot org changed:

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


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

[Bug c++/41275] [4.5 Regression] ICE: expand_expr_real_1, at expr.c:8416

[hjl at gcc dot gnu dot org]

------- Comment #9 from hjl at gcc dot gnu dot org  2009-09-28 20:39 -------
Subject: Bug 41275

Author: hjl
Date: Mon Sep 28 20:38:53 2009
New Revision: 152250

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=152250
Log:
2009-09-28  H.J. Lu  <hongjiu.lu@intel.com>

        Backport from mainline:
        2009-09-26  Andreas Schwab  <schwab@linux-m68k.org>

        PR c/41476
        * gcc.dg/cond-constqual-1.c: New test.

        2009-09-26  Michael Matz  <matz@suse.de>

        PR lto/40758
        PR middle-end/41470
        * gcc.dg/pr41470.c: New test.

        2009-09-23  Alexandre Oliva  <aoliva@redhat.com>

        PR debug/41248
        * gcc.dg/pr41248.c: New.

        2009-09-22  Alexandre Oliva  <aoliva@redhat.com>

        PR debug/41295
        * gcc.dg/pr41295.c: New.

        2009-09-20  Andrew Pinski  <pinskia@gcc.gnu.org>

        PR middle-end/40642
        * g++.dg/torture/pr40642.C: New testcase.

        2009-09-17  Michael Matz  <matz@suse.de>

        PR middle-end/41347
        * gfortran.dg/pr41347.f90: New test.

        2009-09-16  Michael Matz  <matz@suse.de>

        PR fortran/41212
        * gfortran.dg/pr41212.f90: New test.

        2009-09-11  Michael Matz  <matz@suse.de>

        PR middle-end/41275
        * g++.dg/tree-ssa/pr41275.C: New test.

        2009-09-11  Janus Weil  <janus@gcc.gnu.org>

        PR fortran/41242
        * gfortran.dg/proc_ptr_comp_21.f90: New.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/torture/pr40642.C
      - copied unchanged from r152249,
trunk/gcc/testsuite/g++.dg/torture/pr40642.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/tree-ssa/pr41275.C
      - copied unchanged from r152249,
trunk/gcc/testsuite/g++.dg/tree-ssa/pr41275.C
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/cond-constqual-1.c
      - copied unchanged from r152248,
trunk/gcc/testsuite/gcc.dg/cond-constqual-1.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/pr41248.c
      - copied unchanged from r152248, trunk/gcc/testsuite/gcc.dg/pr41248.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/pr41295.c
      - copied unchanged from r152249, trunk/gcc/testsuite/gcc.dg/pr41295.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/pr41470.c
      - copied unchanged from r152248, trunk/gcc/testsuite/gcc.dg/pr41470.c
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/pr41212.f90
      - copied unchanged from r152249,
trunk/gcc/testsuite/gfortran.dg/pr41212.f90
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/pr41347.f90
      - copied unchanged from r152249,
trunk/gcc/testsuite/gfortran.dg/pr41347.f90
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/proc_ptr_comp_21.f90
      - copied unchanged from r152249,
trunk/gcc/testsuite/gfortran.dg/proc_ptr_comp_21.f90
Modified:
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


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