[Bug middle-end/65597] New: ICE in build_outer_var_ref, at omp-low.c:1043

[Bug middle-end/65597] New: ICE in build_outer_var_ref, at omp-low.c:1043

[burnus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65597

            Bug ID: 65597
           Summary: ICE in build_outer_var_ref, at omp-low.c:1043
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Keywords: openmp
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: burnus at gcc dot gnu.org

The following Fortran program causes an ICE with -fopenmp; it doesn't with
-fopenmp-simd nor does the equivalent C program ICE:


 !$omp do simd collapse(1)
 ^
internal compiler error: in build_outer_var_ref, at omp-low.c:1206
0xaa0d0c build_outer_var_ref
        ../../gcc/omp-low.c:1206
0xaae1d1 lower_rec_input_clauses
        ../../gcc/omp-low.c:3959
0xab1c8a lower_omp_for
        ../../gcc/omp-low.c:10622


external foo
!$omp do simd collapse(1)
do i = 1, 151, 31
   call foo();
end do
end

[Bug middle-end/65597] ICE in build_outer_var_ref, at omp-low.c:1043

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65597

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jakub at gcc dot gnu.org

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 35184
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35184&action=edit
gcc5-pr65597.patch

That said, IMHO this patch is a step in the right direction and fixes this
Fortran testcase, though the C testcase as well as similar Fortran testcase
with a different var under linear clause still ICEs.

[Bug middle-end/65597] ICE in build_outer_var_ref, at omp-low.c:1043

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65597

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Mon Mar 30 17:54:05 2015
New Revision: 221776

URL: https://gcc.gnu.org/viewcvs?rev=221776&root=gcc&view=rev
Log:
    PR fortran/65597
    * trans-openmp.c (gfc_trans_omp_do): For !simple simd with explicit
    linear clause for the iterator set OMP_CLAUSE_LINEAR_NO_COPYIN.
    For implcitly added !simple OMP_CLAUSE_LINEAR set it too.  Use step 1
    instead of the original step on the new iterator - count.

    * testsuite/libgomp.fortran/pr65597.f90: New test.

Added:
    trunk/libgomp/testsuite/libgomp.fortran/pr65597.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/trans-openmp.c
    trunk/libgomp/ChangeLog

[Bug middle-end/65597] ICE in build_outer_var_ref, at omp-low.c:1043

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65597

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed Jun  3 15:31:46 2015
New Revision: 224090

URL: https://gcc.gnu.org/viewcvs?rev=224090&root=gcc&view=rev
Log:
        Backported from mainline
        2015-03-30  Jakub Jelinek  <jakub@redhat.com>

        PR fortran/65597
        * trans-openmp.c (gfc_trans_omp_do): For !simple simd with explicit
        linear clause for the iterator set OMP_CLAUSE_LINEAR_NO_COPYIN.
        For implcitly added !simple OMP_CLAUSE_LINEAR set it too.  Use step 1
        instead of the original step on the new iterator - count.

        * testsuite/libgomp.fortran/pr65597.f90: New test.

Added:
    branches/gcc-4_9-branch/libgomp/testsuite/libgomp.fortran/pr65597.f90
Modified:
    branches/gcc-4_9-branch/gcc/fortran/ChangeLog
    branches/gcc-4_9-branch/gcc/fortran/trans-openmp.c
    branches/gcc-4_9-branch/libgomp/ChangeLog

[Bug middle-end/65597] ICE in build_outer_var_ref, at omp-low.c:1043

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65597

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.