[Bug tree-optimization/66129] New: [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[Bug tree-optimization/66129] New: [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[hjl.tools at gmail dot com]

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

            Bug ID: 66129
           Summary: [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c
                    execution test
           Product: gcc
           Version: 6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: rguenther at suse dot de
  Target Milestone: ---

On x86, r223059 caused:

FAIL: gcc.dg/vect/vect-strided-a-mult.c execution test
FAIL: gcc.dg/vect/vect-strided-a-mult.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-a-u16-i2.c execution test
FAIL: gcc.dg/vect/vect-strided-a-u16-i2.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-a-u16-i4.c execution test
FAIL: gcc.dg/vect/vect-strided-a-u16-i4.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-a-u16-mult.c execution test
FAIL: gcc.dg/vect/vect-strided-a-u16-mult.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-a-u32-mult.c execution test
FAIL: gcc.dg/vect/vect-strided-a-u32-mult.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-mult.c execution test
FAIL: gcc.dg/vect/vect-strided-mult.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-mult-char-ls.c execution test
FAIL: gcc.dg/vect/vect-strided-mult-char-ls.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-u16-i2.c execution test
FAIL: gcc.dg/vect/vect-strided-u16-i2.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-u16-i4.c execution test
FAIL: gcc.dg/vect/vect-strided-u16-i4.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-u32-i4.c execution test
FAIL: gcc.dg/vect/vect-strided-u32-i4.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-u32-i8.c execution test
FAIL: gcc.dg/vect/vect-strided-u32-i8.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-strided-u32-mult.c execution test
FAIL: gcc.dg/vect/vect-strided-u32-mult.c -flto -ffat-lto-objects execution
test
FAIL: gcc.dg/vect/vect-strided-u8-i2.c execution test
FAIL: gcc.dg/vect/vect-strided-u8-i2.c -flto -ffat-lto-objects execution test

with

make check-gcc RUNTESTFLAGS="vect.exp=vect-strided*.c
--target_board='unix{-march=corei7}'"

[Bug tree-optimization/66129] [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-05-13
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org
   Target Milestone|---                         |6.0
     Ever confirmed|0                           |1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Mine.

[Bug tree-optimization/66129] [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[rguenth at gcc dot gnu.org]

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

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Oops...

Index: tree-vect-slp.c
===================================================================
--- tree-vect-slp.c     (revision 223119)
+++ tree-vect-slp.c     (working copy)
@@ -3459,7 +3459,7 @@ vect_schedule_slp_instance (slp_tree nod
              tree *melts = XALLOCAVEC (tree, TYPE_VECTOR_SUBPARTS (vectype));
              for (l = 0; l < TYPE_VECTOR_SUBPARTS (vectype); ++l)
                {
-                 if (k > group_size)
+                 if (k >= group_size)
                    k = 0;
                  melts[l] = build_int_cst
                      (meltype, mask[k++] * TYPE_VECTOR_SUBPARTS (vectype) +
l);

[Bug tree-optimization/66129] [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[rguenth at gcc dot gnu.org]

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

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Wed May 13 14:38:19 2015
New Revision: 223164

URL: https://gcc.gnu.org/viewcvs?rev=223164&root=gcc&view=rev
Log:
2015-05-13  Richard Biener  <rguenther@suse.de>

        PR tree-optimization/66129
        * tree-vect-slp.c (vect_build_slp_tree): Make sure all ops are
        commutative.
        (vect_schedule_slp_instance): Fix typo.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree-vect-slp.c

[Bug tree-optimization/66129] [6 Regression] FAIL: gcc.dg/vect/vect-strided-*c execution test

[rguenth at gcc dot gnu.org]

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

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

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

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.