[Bug tree-optimization/43097] New: ICE in rename_map_elt_info for Graphite, vect in 173.applu

[Bug tree-optimization/43097] New: ICE in rename_map_elt_info for Graphite, vect in 173.applu

[janis at gcc dot gnu dot org]

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

GCC trunk gets an internal compiler error when building SPEC CPU2000 test
173.applu on powerpc64-linux with options "-O2 -fgraphite-identity
-ftree-vectorize" for either -m32 or -m64.  Here's a minimized testcase:

      subroutine foo (ldmx,ldmy,nx,ny,v)
      implicit real*8 (a-h, o-z)
      dimension v(5,ldmx,ldmy,*)
      dimension tmat(5,5)

      k = 2
      do j = 2, ny-1
         do i = 2, nx-1
            do ip = 1, 4
               do m = ip+1, 5
                  v(m,i,j,k) = v(m,i,j,k) * m
               end do
            end do
            do m = 5, 1, -1
               do l = m+1, 5
                  v(m,i,j,k) = v(l,i,j,k)
               end do
               v(m,i,j,k) = m
           end do
         end do
      end do
      return
      end

elm3b149% /home/janis/tools/gcc-trunk-anonsvn/bin/gfortran -c -O2
-fgraphite-identity -ftree-vectorize bug.f
bug.f: In function ‘foo’:
bug.f:1:0: internal compiler error: tree check: expected ssa_name, have
var_decl in rename_map_elt_info, at sese.c:81
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

The benchmark and testcase compile cleanly with GCC 4.4.2.  The failure
begins with this patch:

    http://gcc.gnu.org/viewcvs?view=rev&rev=156711

    r156711 | spop | 2010-02-11 19:42:38 +0000 (Thu, 11 Feb 2010)


-- 
           Summary: ICE in rename_map_elt_info for Graphite, vect in
                    173.applu
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: janis at gcc dot gnu dot org
GCC target triplet: powerpc64-linux


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

[Bug tree-optimization/43097] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[dominiq at lps dot ens dot fr]

------- Comment #1 from dominiq at lps dot ens dot fr  2010-02-16 18:18 -------
Confirmed on x86_64-apple-darwin10 revision 156811.


-- 


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

[Bug tree-optimization/43097] [4.5 Regression] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|ICE in rename_map_elt_info  |[4.5 Regression] ICE in
                   |for Graphite, vect in       |rename_map_elt_info for
                   |173.applu                   |Graphite, vect in 173.applu
   Target Milestone|---                         |4.5.0


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

[Bug tree-optimization/43097] [4.5 Regression] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2


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

[Bug tree-optimization/43097] [4.5 Regression] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2010-02-18 18:36 -------
Happens on x86_64-linux-gnu also even without -ftree-vectorize.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
 GCC target triplet|powerpc64-linux             |
   Last reconfirmed|0000-00-00 00:00:00         |2010-02-18 18:36:56
               date|                            |


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

[Bug tree-optimization/43097] [4.5 Regression] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[spop at gcc dot gnu dot org]

------- Comment #3 from spop at gcc dot gnu dot org  2010-02-22 15:19 -------
This should have been added to this PR:
I copied and forgot to adjust the PR number.


Author: spop
Date: Mon Feb 22 15:13:39 2010
New Revision: 156964

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156964
Log:
Fix PR43097: rename only SSA_NAMEs.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>

       PR middle-end/43140
       * sese.c (get_rename): Assert that old_name is an SSA_NAME.
       (rename_variables_in_stmt): Continue when the use is not an SSA_NAME.

       * gfortran.dg/graphite/pr43097.f: New.

Added:
   branches/graphite/gcc/testsuite/gfortran.dg/graphite/pr43097.f
Modified:
   branches/graphite/gcc/ChangeLog.graphite
   branches/graphite/gcc/sese.c


-- 

spop at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |spop at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2010-02-18 18:36:56         |2010-02-22 15:19:27
               date|                            |


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

[Bug tree-optimization/43097] [4.5 Regression] ICE in rename_map_elt_info for Graphite, vect in 173.applu

[spop at gcc dot gnu dot org]

------- Comment #4 from spop at gcc dot gnu dot org  2010-02-23 13:06 -------
Fixed on trunk with this commit:

Author: spop
Date: Tue Feb 23 12:59:17 2010
New Revision: 156995

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156995
Log:
Fix PR43097: rename only SSA_NAMEs.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>

       PR middle-end/43140
       * sese.c (get_rename): Assert that old_name is an SSA_NAME.
       (rename_variables_in_stmt): Continue when the use is not an SSA_NAME.

       * gfortran.dg/graphite/pr43097.f: New.

Added:
   trunk/gcc/testsuite/gfortran.dg/graphite/pr43097.f
Modified:
   trunk/gcc/ChangeLog.graphite
   trunk/gcc/sese.c


-- 

spop at gcc dot gnu dot org changed:

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


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