[Bug libgomp/35185] New: ICE using openmp with g++-4.2

[Bug libgomp/35185] New: ICE using openmp with g++-4.2

[edgurgel at gmail dot com]

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

System: Ubuntu 7.10
gcc version: 4.2.0

Problem: Using openmp extension

erro interno de compilação: in create_tmp_var, at gimplify.c:487


-- 
           Summary: ICE using openmp with g++-4.2
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libgomp
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: edgurgel at gmail dot com


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

[Bug libgomp/35185] ICE using openmp with g++-4.2

[edgurgel at gmail dot com]

------- Comment #1 from edgurgel at gmail dot com  2008-02-13 18:30 -------
Created an attachment (id=15142)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15142&action=view)
source


-- 


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

[Bug libgomp/35185] ICE using openmp with g++-4.2

[edgurgel at gmail dot com]

------- Comment #2 from edgurgel at gmail dot com  2008-02-13 18:30 -------
Created an attachment (id=15143)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15143&action=view)
header


-- 


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

[Bug libgomp/35185] ICE using openmp with g++-4.2

[edgurgel at gmail dot com]

------- Comment #3 from edgurgel at gmail dot com  2008-02-13 18:31 -------
Created an attachment (id=15144)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15144&action=view)
.i


-- 


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

[Bug libgomp/35185] ICE using openmp with g++-4.2

[edgurgel at gmail dot com]

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



------- Comment #4 from edgurgel at gmail dot com  2008-02-13 18:33 -------
G++ gives

raycaster.cpp: In member function ‘void Raycaster::cast() const’:
raycaster.cpp:43: erro interno de compilação: in create_tmp_var, at
gimplify.c:487
Por favor, envie um relato completo de erro,
com o código pré-processado se possível.
Veja <URL:http://gcc.gnu.org/bugs.html> para instruções.
For Debian GNU/Linux specific bug reporting instructions,
see <URL:file:///usr/share/doc/gcc-4.2/README.Bugs>.
make: ** [raycaster.o] Erro 1

trying to compile raycaster.cpp with g++-4.2 -c -O3 -pipe -mmmx -m3dnow -msse2
-msse  -fopenmp -save-temps raycaster.cpp -o raycaster.o

on ubuntu 7.10


-- 


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #5 from jakub at gcc dot gnu dot org  2008-02-15 18:58 -------
Simplified testcase:
struct S
{
  S ();
  ~S ();
  int s;
};

void bar (S);

void
foo ()
{
  S s;
#pragma omp parallel shared (s)
  bar (s);
}

Works if s is firstprivate or private, or if it doesn't have user defined
destructor.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2008-02-15 18:58:42
               date|                            |


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #6 from jakub at gcc dot gnu dot org  2008-02-15 21:07 -------
The problem is the regimplifying done by lower_omp_1.
We have
D.NNNN = s;
where both of the vars have RECORD_TYPE with TREE_ADDRESSABLE types.
lower_omp_1 sees on the right hand side VAR_DECL with DECL_VALUE_EXPR
(wi->is_lhs = false, wi->val_only = true), which is replaced with
*.omp_data_i->s which doesn't satisfy is_gimple_val predicate lower_regimplify
uses and of course the compiler isn't allowed to create temporaries of
TREE_ADDRESSABLE types on its own.  In this case I believe we need to force
the .omp_data_i->s address into a temporary first and then *temp should
hopefully satisfy the predicate.  Diego, any ideas?


-- 

jakub 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=35185

[Bug middle-end/35185] ICE using openmp with g++-4.2

[rguenth at gcc dot gnu dot org]

------- Comment #7 from rguenth at gcc dot gnu dot org  2008-02-18 09:41 -------
*** Bug 35247 has been marked as a duplicate of this bug. ***


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ttsiod at softlab dot ntua
                   |                            |dot gr


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #8 from jakub at gcc dot gnu dot org  2008-03-12 17:49 -------
Created an attachment (id=15305)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15305&action=view)
gcc44-pr35185.patch

WIP patch I'm playing with.  It just looks for statements that reference
variables with DECL_HAS_VALUE_EXPR_P and regimplifies them as whole, rather
than
only regimplifying parts of them.
I still get 4 libgomp regressions:
FAIL: libgomp.c/nestedfn-4.c execution test
FAIL: libgomp.c/pr33880.c execution test
FAIL: libgomp.c/pr35130.c execution test
FAIL: libgomp.fortran/pr35130.f90  -O  execution test
and most likely further work will be needed for -ftree-parallelize-loops
where lower_omp is called on SSA.


-- 


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2008-02-15 18:58:42         |2008-03-12 21:23:57
               date|                            |


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #9 from jakub at gcc dot gnu dot org  2008-03-13 09:27 -------
Subject: Bug 35185

Author: jakub
Date: Thu Mar 13 09:26:25 2008
New Revision: 133162

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=133162
Log:
        PR middle-end/35185
        * omp-low.c (lower_regimplify, init_tmp_var, save_tmp_var): Removed.
        (lower_omp_2): New function.
        (lower_omp_1, lower_omp): Rewritten.

        * testsuite/libgomp.c++/pr35185.C: New test.

Added:
    trunk/libgomp/testsuite/libgomp.c++/pr35185.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/omp-low.c
    trunk/libgomp/ChangeLog


-- 


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #11 from jakub at gcc dot gnu dot org  2008-03-13 09:59 -------
Fixed for 4.3.1 and later.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
      Known to work|                            |4.3.1 4.4.0
         Resolution|                            |FIXED
   Target Milestone|---                         |4.3.1


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

[Bug middle-end/35185] ICE using openmp with g++-4.2

[jakub at gcc dot gnu dot org]

------- Comment #10 from jakub at gcc dot gnu dot org  2008-03-13 09:58 -------
Subject: Bug 35185

Author: jakub
Date: Thu Mar 13 09:57:34 2008
New Revision: 133165

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=133165
Log:
        PR middle-end/35185
        * omp-low.c (lower_regimplify, init_tmp_var, save_tmp_var): Removed.
        (lower_omp_2): New function.
        (lower_omp_1, lower_omp): Rewritten.

        * testsuite/libgomp.c++/pr35185.C: New test.

Added:
    branches/gcc-4_3-branch/libgomp/testsuite/libgomp.c++/pr35185.C
      - copied unchanged from r133162,
trunk/libgomp/testsuite/libgomp.c++/pr35185.C
Modified:
    branches/gcc-4_3-branch/gcc/ChangeLog
    branches/gcc-4_3-branch/gcc/omp-low.c
    branches/gcc-4_3-branch/libgomp/ChangeLog


-- 


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