[Bug target/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[Bug target/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[irar at il dot ibm.com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |irar at il dot ibm.com
      Known to work|                            |
      Known to fail|                            |

--- Comment #3 from Ira Rosen <irar at il dot ibm.com> 2010-10-03 10:24:57 UTC ---
I can't reproduce it. Please attach vectorizer's dump
(-fdump-tree-vect-details).

Thanks,
Ira

[Bug target/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[irar at il dot ibm.com]

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

--- Comment #5 from Ira Rosen <irar at il dot ibm.com> 2010-10-04 12:39:59 UTC ---
Thanks, -m32 did the trick.

I am testing a patch.

Thanks,
Ira

[Bug tree-optimization/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[irar at gcc dot gnu.org]

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

--- Comment #6 from irar at gcc dot gnu.org 2010-10-05 12:01:32 UTC ---
Author: irar
Date: Tue Oct  5 12:01:29 2010
New Revision: 164987

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

    PR tree-optimization/45752
    * tree-vect-slp.c (vect_get_mask_element): Remove static
    variables, make them function arguments.
    (vect_transform_slp_perm_load): Pass new arguments to
    vect_get_mask_element.


Added:
    trunk/gcc/testsuite/gcc.dg/vect/pr45752.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-vect-slp.c

[Bug tree-optimization/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[irar at gcc dot gnu.org]

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

--- Comment #7 from irar at gcc dot gnu.org 2010-10-05 12:25:00 UTC ---
Author: irar
Date: Tue Oct  5 12:24:43 2010
New Revision: 164989

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

    PR tree-optimization/45752
    * tree-vect-slp.c (vect_get_mask_element): Remove static
    variables, make them function arguments.
    (vect_transform_slp_perm_load): Pass new arguments to
    vect_get_mask_element.


Added:
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/vect/pr45752.c
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_5-branch/gcc/tree-vect-slp.c

[Bug tree-optimization/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[irar at il dot ibm.com]

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

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

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

--- Comment #8 from Ira Rosen <irar at il dot ibm.com> 2010-10-07 07:07:44 UTC ---
Fixed.

[Bug target/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-09-23 09:30 -------
Confirmed.

#2  0x0000000000ec5b41 in ix86_vectorize_builtin_vec_perm_ok (
    vec_type=0x7ffff5a04dc8, mask=0x7ffff5a7d2a0)
    at /space/rguenther/src/svn/gcc-4_5-branch/gcc/config/i386/i386.c:30144
30144     gcc_assert (vec_mask > 0 && vec_mask <= 3);
(gdb) p vec_mask
$1 = 0

1912                      if (!targetm.vectorize.builtin_vec_perm_ok (vectype,
1913                                                                 
mask_vec))
(gdb) call debug_generic_expr (mask_vec)
{ 0xfffffffffffffffffffffffffffffff8, 0xfffffffffffffffffffffffffffffff8,
0xfffffffffffffffffffffffffffffff8, 0xfffffffffffffffffffffffffffffffb,
0xfffffffffffffffffffffffffffffffb, 0xfffffffffffffffffffffffffffffffb,
0xfffffffffffffffffffffffffffffffe, 0xfffffffffffffffffffffffffffffffe }
(gdb) call debug_generic_expr (vectype)
const vector short unsigned int

I don't know why the mask elements are negative, but that's certainly
the problem with extract_vec_perm_cst which requires positive elements.

The problem is surely latent on trunk (no changes in that area).

Ira?


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |irar at gcc dot gnu dot org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
      Known to fail|                            |4.5.0 4.5.1
      Known to work|                            |4.4.4 4.6.0
   Last reconfirmed|0000-00-00 00:00:00         |2010-09-23 09:30:22
               date|                            |


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

[Bug target/45752] [4.5 regression] ICE in ix86_vectorize_builtin_vec_perm_ok

[hjl dot tools at gmail dot com]

------- Comment #1 from hjl dot tools at gmail dot com  2010-09-23 07:20 -------
It is caused by revision 155584:

http://gcc.gnu.org/ml/gcc-cvs/2010-01/msg00043.html

and fixed by revision 161655:

http://gcc.gnu.org/ml/gcc-cvs/2010-07/msg00006.html

on trunk.


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rguenth at gcc dot gnu dot
                   |                            |org
   Target Milestone|---                         |4.5.2


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