[Bug c/51015] New: ice in vect_determine_vectorization_factor with -O3

[Bug c/51015] New: ice in vect_determine_vectorization_factor with -O3

[dcb314 at hotmail dot com]

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

             Bug #: 51015
           Summary: ice in vect_determine_vectorization_factor with -O3
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: dcb314@hotmail.com


Created attachment 25745
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25745
C source code

I just tried to compile the package e2fsprogs-1.42-0.5
on latest trunk snapshot 20111105 on an AMD x86_64 box.

The compiler said

pass1.c:543:6: internal compiler error: in vect_determine_vectorization_factor,
at tree-vect-loop.c:348
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

Preprocessed source code attached. Flag -O3 required.

[Bug c/51015] [4.7 Regression] ice in vect_determine_vectorization_factor with -O3

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011-11-07
                 CC|                            |ira at gcc dot gnu.org,
                   |                            |jakub at redhat dot com
            Summary|ice in                      |[4.7 Regression] ice in
                   |vect_determine_vectorizatio |vect_determine_vectorizatio
                   |n_factor with -O3           |n_factor with -O3
     Ever Confirmed|0                           |1

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2011-11-07 23:48:01 UTC ---
Reduced test:

typedef unsigned long long __u64;
static __u64 ext2_max_sizes[16 -
       10 + 1];

void e2fsck_pass1()
{
 int i;
 __u64 max_sizes;

 for (i = 10; i <= 16; i++) {
  max_sizes = 12 + (1ULL << ((i) - 2));
  max_sizes = max_sizes + (1ULL << ((i) - 2)) * (1ULL << ((i) - 2));
  max_sizes = max_sizes + (1ULL << ((i) - 2)) * (1ULL << ((i) - 2)) * (1ULL <<
((i) - 2));
  ext2_max_sizes[i - 10] = max_sizes;
 }

}

r179960 is OK
r180683 gives the ICE

[Bug c/51015] [4.7 Regression] ice in vect_determine_vectorization_factor with -O3

[irar at il dot ibm.com]

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

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |irar at il dot ibm.com
         AssignedTo|unassigned at gcc dot       |irar at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #2 from Ira Rosen <irar at il dot ibm.com> 2011-11-08 10:22:53 UTC ---
Some of the patterns added lately use pattern def stmt and set its vectype,
hence the assert should be updated:

Index: tree-vect-loop.c
===================================================================
--- tree-vect-loop.c    (revision 181150)
+++ tree-vect-loop.c    (working copy)
@@ -345,7 +345,8 @@ vect_determine_vectorization_factor (loo
                 that contain a dataref, or for "pattern-stmts" (stmts
generated
                 by the vectorizer to represent/replace a certain idiom).  */
              gcc_assert (STMT_VINFO_DATA_REF (stmt_info)
-                         || is_pattern_stmt_p (stmt_info));
+                         || is_pattern_stmt_p (stmt_info)
+                         || pattern_def);
              vectype = STMT_VINFO_VECTYPE (stmt_info);
            }
          else

[Bug tree-optimization/51015] [4.7 Regression] ice in vect_determine_vectorization_factor with -O3

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.0

[Bug tree-optimization/51015] [4.7 Regression] ice in vect_determine_vectorization_factor with -O3

[irar at gcc dot gnu.org]

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

--- Comment #3 from irar at gcc dot gnu.org 2011-11-09 06:42:07 UTC ---
Author: irar
Date: Wed Nov  9 06:41:59 2011
New Revision: 181191

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181191
Log:

        PR tree-optimization/51015
        * tree-vect-loop.c (vect_determine_vectorization_factor): Expect
        vectype to be set for pattern def stmts.


Added:
    trunk/gcc/testsuite/gcc.dg/vect/pr51015.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-vect-loop.c

[Bug tree-optimization/51015] [4.7 Regression] ice in vect_determine_vectorization_factor with -O3

[irar at il dot ibm.com]

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

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|4.7.0                       |---

--- Comment #4 from Ira Rosen <irar at il dot ibm.com> 2011-11-09 07:54:16 UTC ---
Fixed.