[Bug c++/26084] New: [gomp-branch] ICE (segfault) on C++ OpenMP coce

[Bug c++/26084] New: [gomp-branch] ICE (segfault) on C++ OpenMP coce

[martin at mpa-garching dot mpg dot de]

New ICE on the gomp branch.

~/data/planck/LevelS>g++ -v -fopenmp -c bug.ii
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: /scratch/gompcc/configure --quiet --prefix=/scratch/ugccgomp
--enable-languages=c++,fortran --with-gmp=/usr/local/appl/gmp-4.1.4
--enable-checking=release
Thread model: posix
gcc version 4.2.0-gomp-20050608-branch 20060202 (experimental) (merged
20060202)

/scratch/ugccgomp/libexec/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/cc1plus
-fpreprocessed bug.ii -quiet -dumpbase bug.ii -mtune=generic -auxbase bug
-version -fopenmp -o /tmp/ccQCx6GI.s
GNU C++ version 4.2.0-gomp-20050608-branch 20060202 (experimental) (merged
20060202) (i686-pc-linux-gnu)
        compiled by GNU C version 4.2.0-gomp-20050608-branch 20060202
(experimental) (merged 20060202).
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: c854429a5b3be01718b61b454f3d31d6
/afs/mpa/data/martin/planck/LevelS/Healpix_cxx/alm_map_tools.cc: In function
'void
_Z7map2almIfEvRKSt6vectorI8ringpairSaIS1_EEPKT_R3AlmI8xcomplexIS6_EEb.omp_fn.19(void*)':
/afs/mpa/data/martin/planck/LevelS/Healpix_cxx/alm_map_tools.cc:257: internal
compiler error: Segmentation fault
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.


-- 
           Summary: [gomp-branch] ICE (segfault) on C++ OpenMP coce
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: martin at mpa-garching dot mpg dot de
 GCC build triplet: i686-pc-linux-gnu
  GCC host triplet: i686-pc-linux-gnu
GCC target triplet: i686-pc-linux-gnu


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

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP coce

[martin at mpa-garching dot mpg dot de]

------- Comment #1 from martin at mpa-garching dot mpg dot de  2006-02-02 23:48 -------
Created an attachment (id=10774)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10774&action=view)
unreduced test case


-- 


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

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP coce

[martin at mpa-garching dot mpg dot de]

------- Comment #2 from martin at mpa-garching dot mpg dot de  2006-02-04 11:31 -------
Reduced testcase:

template <typename T> struct arr {
  long s;
  T *d;

  arr(long sz) : s(sz), d (s>0 ? new T[s] : 0) {}
  ~arr() { delete[] d; }
  T &operator[] (int n)  {return d[n];}
  };

void map2alm (const double *map) {
#pragma omp parallel
{
    arr<double> Ylm(1), alm_tmp(1);
    Ylm[1] +=2;
}
  }


~/tmp>g++ -I ~/data/planck/LevelS/linux_gcc/include bug.cc -fopenmp -c -v
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: /scratch/gompcc/configure --quiet --prefix=/scratch/ugccgomp
--enable-languages=c++,fortran --with-gmp=/usr/local/appl/gmp-4.1.4
--enable-checking=release
Thread model: posix
gcc version 4.2.0-gomp-20050608-branch 20060202 (experimental) (merged
20060202)

/scratch/ugccgomp/libexec/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/cc1plus
-quiet -v -I /afs/mpa/home/martin/data/planck/LevelS/linux_gcc/include
-D_GNU_SOURCE -D_REENTRANT bug.cc -quiet -dumpbase bug.cc -mtune=generic
-auxbase bug -version -fopenmp -o /tmp/ccYgPhms.s
ignoring nonexistent directory
"/scratch/ugccgomp/lib/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/../../../../i686-pc-linux-gnu/include"
#include "..." search starts here:
#include <...> search starts here:
 /afs/mpa/home/martin/data/planck/LevelS/linux_gcc/include

/scratch/ugccgomp/lib/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/../../../../include/c++/4.2.0-gomp-20050608-branch

/scratch/ugccgomp/lib/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/../../../../include/c++/4.2.0-gomp-20050608-branch/i686-pc-linux-gnu

/scratch/ugccgomp/lib/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/../../../../include/c++/4.2.0-gomp-20050608-branch/backward
 /usr/local/include
 /scratch/ugccgomp/include
 /scratch/ugccgomp/lib/gcc/i686-pc-linux-gnu/4.2.0-gomp-20050608-branch/include
 /usr/include
End of search list.
GNU C++ version 4.2.0-gomp-20050608-branch 20060202 (experimental) (merged
20060202) (i686-pc-linux-gnu)
        compiled by GNU C version 4.2.0-gomp-20050608-branch 20060202
(experimental) (merged 20060202).
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: c854429a5b3be01718b61b454f3d31d6
bug.cc: In function 'void _Z7map2almPKd.omp_fn.0(void*)':
bug.cc:11: internal compiler error: Segmentation fault
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=26084

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP code

[reichelt at gcc dot gnu dot org]

------- Comment #3 from reichelt at gcc dot gnu dot org  2006-02-06 09:40 -------
I don't get a segfault, but the error message:

bug.cc:11: internal compiler error: vector VEC(eh_region,base) index domain
error, in can_throw_internal_1 at except.c:2580

Looks like a duplicate of PR 26076 to me.


-- 

reichelt at gcc dot gnu dot org changed:

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


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

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP code

[martin at mpa-garching dot mpg dot de]

------- Comment #4 from martin at mpa-garching dot mpg dot de  2006-02-06 11:15 -------
(In reply to comment #3)
> I don't get a segfault, but the error message:
> 
> bug.cc:11: internal compiler error: vector VEC(eh_region,base) index domain
> error, in can_throw_internal_1 at except.c:2580

I still get the segmentation fault. Probably because I configured with
--enable-checking=release...


-- 


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

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP code

[martin at mpa-garching dot mpg dot de]

------- Comment #5 from martin at mpa-garching dot mpg dot de  2006-02-07 10:44 -------
(In reply to comment #4)

> I still get the segmentation fault. Probably because I configured with
> --enable-checking=release...

I can confirm that I get the same error message reported Volker if I boostrap
gcc without "--enable-checking=release".


-- 


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

[Bug c++/26084] [gomp-branch] ICE (segfault) on C++ OpenMP code

[dnovillo at gcc dot gnu dot org]

------- Comment #6 from dnovillo at gcc dot gnu dot org  2006-02-07 12:12 -------

Mine.


-- 

dnovillo at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |dnovillo at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-02-07 12:12:34
               date|                            |


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2006-03-09 18:49 -------
PR 26076 has a reducer testcase and I can reduce the testcase in here into that
one.

*** This bug has been marked as a duplicate of 26076 ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |DUPLICATE
            Summary|[gomp-branch] ICE (segfault)|ICE (segfault) on C++ OpenMP
                   |on C++ OpenMP code          |code


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[dnovillo at gcc dot gnu dot org]

------- Comment #8 from dnovillo at gcc dot gnu dot org  2006-03-09 19:01 -------

Andrew, please do not close bugs if you do not understand the problem.  You
obviously have not even tried to understand the problem.  This PR is related to
26076 but it is *not* a duplicate.

The patch that I developed originally fixes the reduced cases in this PR and
26076, but it DOES NOT fix the unreduced test case.

There are more interactions with EH than what my initial patch fixed.  Richard
is looking at fixing the original, unreduced case in this PR.


-- 

dnovillo at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rth at gcc dot gnu dot org
  BugsThisDependsOn|26076                       |
             Status|RESOLVED                    |REOPENED
         Resolution|DUPLICATE                   |


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[pinskia at gcc dot gnu dot org]

------- Comment #9 from pinskia at gcc dot gnu dot org  2006-03-09 19:03 -------
(In reply to comment #8)
> Andrew, please do not close bugs if you do not understand the problem.  You
> obviously have not even tried to understand the problem.  This PR is related to
> 26076 but it is *not* a duplicate.

Then there are two issues, one for the reduced testcase which is PR 26076 and
another issue.  But please don't say I don't understand the issue when the fact
is this testcase here actually reduces to the testcase in PR 26076.


-- 


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[dnovillo at redhat dot com]

------- Comment #10 from dnovillo at redhat dot com  2006-03-09 19:10 -------
Subject: Re:  ICE (segfault) on C++ OpenMP code

On 03/09/06 14:03, pinskia at gcc dot gnu dot org wrote:

> Then there are two issues, one for the reduced testcase which is PR 26076 and
> another issue.  But please don't say I don't understand the issue when the fact
> is this testcase here actually reduces to the testcase in PR 26076.
> 
The fact that you closed one as a duplicate of the other is a clear
indication that you have no idea what the issue is about.  Reducing one
test case to another one means nothing.  The very reduction removes
information that hides other bugs, or other aspects of the same bug.

If you are so keen on closing PRs, close 26076 as a duplicate of this one.


-- 


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[pinskia at gcc dot gnu dot org]

------- Comment #11 from pinskia at gcc dot gnu dot org  2006-03-09 19:17 -------
(In reply to comment #10)
> If you are so keen on closing PRs, close 26076 as a duplicate of this one.

If you had written a status on this bug before I closed it as a dup, I would
not have closed it as a dup because I would have read that it was not fully a
dup.  Also really a secondary bug should be opened for the other bug you found
after fixing the reduced testcase with a new reduced testcase instead of not
just agrueeing in this bug.  There was no way to know what was going on without
status of the bug in the bug comments itself.  (Note this applies for all the
bug assigned to people).


-- 


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[dnovillo at redhat dot com]

------- Comment #12 from dnovillo at redhat dot com  2006-03-09 19:28 -------
Subject: Re:  ICE (segfault) on C++ OpenMP code

On 03/09/06 14:17, pinskia at gcc dot gnu dot org wrote:

> If you had written a status on this bug before I closed it as a dup,
> I would not have closed it as a dup because I would have read that it
> was not fully a dup.
> 
When in doubt, ask.  I don't have the time you seem to have to keep a
perfectly up-to-date status on the PRs I have in my queue.

I understand that you as a bugmaster would like to minimize open PRs and
have perfect information on all of them, but such is life.  It's an
imperfect system and the bugs will not always have all the information
you want in them.

Unless you are absolutely certain (i.e., you have worked on a patch or
debugged the problem in detail), you should ask the developer who was
last working on the bug to see what the status is.


-- 


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

[Bug c++/26084] ICE (segfault) on C++ OpenMP code

[rth at gcc dot gnu dot org]

------- Comment #13 from rth at gcc dot gnu dot org  2006-03-22 12:33 -------
Mine.


-- 

rth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|dnovillo at gcc dot gnu dot |rth at gcc dot gnu dot org
                   |org                         |
             Status|REOPENED                    |ASSIGNED
   Last reconfirmed|2006-02-07 12:12:34         |2006-03-22 12:33:27
               date|                            |


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

[Bug middle-end/26084] ICE (segfault) on C++ OpenMP code

[rth at gcc dot gnu dot org]

------- Comment #14 from rth at gcc dot gnu dot org  2006-03-22 12:50 -------
Subject: Bug 26084

Author: rth
Date: Wed Mar 22 12:50:45 2006
New Revision: 112283

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=112283
Log:
        PR middle-end/26084
        * except.c (duplicate_eh_regions_0): New.
        (duplicate_eh_region_1): Duplicate the children of the node as
        well as the node itself.  Link them up properly.
        (duplicate_eh_region_2): Merge into ...
        (duplicate_eh_regions): ... here.  Take copy_region argument, and
        copy only a sub-tree if asked.  Simplify copying and fixup.
        (eh_region_outer_p): New.
        * except.h (duplicate_eh_regions): Update decl.
        (eh_region_outer_p): Declare.
        * omp-low.c (lower_omp_single): Fix eh region placement wrt OMP_RETURN.
        (lower_omp_master): Likewise.
        (lower_omp_ordered): Likewise.
        * tree-cfg.c (struct move_stmt_d): Add new_label_map.
        (move_stmt_r): Use it to remap labels.  Handle recursion vs
        remap_decls_p properly.
        (move_block_to_fn): Pass in new_label_map.  Remap RESX_EXPR.
        (find_outermost_region_in_block): New.
        (new_label_mapper): New.
        (move_sese_region_to_fn): Copy eh information to the new function
        properly.
        * tree-inline.c (copy_cfg_body): Update for new duplicate_eh_regions
        argument.
        * tree-pretty-print.c (dump_generic_node): Dump RESX_EXPR region
        number.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/except.c
    trunk/gcc/except.h
    trunk/gcc/omp-low.c
    trunk/gcc/tree-cfg.c
    trunk/gcc/tree-inline.c
    trunk/gcc/tree-pretty-print.c


-- 


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

[Bug middle-end/26084] ICE (segfault) on C++ OpenMP code

[rth at gcc dot gnu dot org]

------- Comment #15 from rth at gcc dot gnu dot org  2006-03-22 12:53 -------
Fixed.


-- 

rth at gcc dot gnu dot org changed:

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


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

[Bug middle-end/26084] ICE (segfault) on C++ OpenMP code

[martin at mpa-garching dot mpg dot de]

------- Comment #16 from martin at mpa-garching dot mpg dot de  2006-03-22 14:37 -------
(In reply to comment #15)
> Fixed.

Unfortunately the unreduced testcase still fails with -O:

/scratch>g++ -O -v -fopenmp bug.ii
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: /scratch/gcc/configure --quiet
--prefix=/afs/mpa/data/martin/ugcc --enable-languages=c++,fortran
--with-gmp=/usr/local/appl/gmp-4.1.4 --enable-checking=release
--without-makeinfo --disable-tls
Thread model: posix
gcc version 4.2.0 20060322 (experimental)
 /afs/mpa/data/martin/ugcc/libexec/gcc/i686-pc-linux-gnu/4.2.0/cc1plus
-fpreprocessed bug.ii -quiet -dumpbase bug.ii -mtune=generic -auxbase bug -O
-version -fopenmp -o /tmp/ccaWAsuo.s
GNU C++ version 4.2.0 20060322 (experimental) (i686-pc-linux-gnu)
        compiled by GNU C version 4.2.0 20060322 (experimental).
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: 3f3a7467532fa739fdc8d9d9e4c52ff9
/afs/mpa/data/martin/planck/LevelS/Healpix_cxx/alm_map_tools.cc: In function
'void alm2map_der1(const Alm<xcomplex<T> >&, const std::vector<ringpair,
std::allocator<ringpair> >&, T*, T*, T*) [with T = float]':
/afs/mpa/data/martin/planck/LevelS/Healpix_cxx/alm_map_tools.cc:644: internal
compiler error: in add_stmt_to_eh_region_fn, at tree-eh.c:100
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.

Should I open a new PR, or should this one be reopened?


-- 


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

[Bug middle-end/26084] ICE (segfault) on C++ OpenMP code

[reichelt at gcc dot gnu dot org]

------- Comment #17 from reichelt at gcc dot gnu dot org  2006-03-23 11:36 -------
The ICE in add_stmt_to_eh_region_fn is now tracked in PR26823.


-- 


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