[Bug middle-end/45706] New: [4.6 regression] gcc.dg/vect/vect-114.c

[Bug middle-end/45706] New: [4.6 regression] gcc.dg/vect/vect-114.c

[hjl dot tools at gmail dot com]

On Linux/ia32, revision 164369 gave

FAIL: gcc.dg/vect/vect-114.c scan-tree-dump-times vect "vectorized 0 loops" 1

Revision 164366 is OK. It may be caused by revision 164367:

http://gcc.gnu.org/ml/gcc-cvs/2010-09/msg00661.html


-- 
           Summary: [4.6 regression] gcc.dg/vect/vect-114.c
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl dot tools at gmail dot com


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[matz at gcc dot gnu dot org]

------- Comment #1 from matz at gcc dot gnu dot org  2010-09-17 16:12 -------
This passes for me, even in -m32 mode (if -msse is added, like vect.exp
does):

% ./cc1 -ftree-vectorize -fno-vect-cost-model -msse2 -O2 \
  vect-114.c -ftree-vectorizer-verbose=2 2>&1 | grep note:
vect-114.c:13: note: LOOP VECTORIZED.
vect-114.c:6: note: vectorized 1 loops in function.


-- 

matz at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.6 regression]            |[4.6 regression]
                   |gcc.dg/vect/vect-114.c      |gcc.dg/vect/vect-114.c


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[hjl dot tools at gmail dot com]

------- Comment #2 from hjl dot tools at gmail dot com  2010-09-17 16:30 -------
(In reply to comment #1)
> This passes for me, even in -m32 mode (if -msse is added, like vect.exp
> does):
> 
> % ./cc1 -ftree-vectorize -fno-vect-cost-model -msse2 -O2 \
>   vect-114.c -ftree-vectorizer-verbose=2 2>&1 | grep note:
> vect-114.c:13: note: LOOP VECTORIZED.
> vect-114.c:6: note: vectorized 1 loops in function.
> 

Please note. The failure is

FAIL: gcc.dg/vect/vect-114.c scan-tree-dump-times vect "vectorized 0 loops" 1
                                                        ^^^^^^^^^^^^^^^^^^^


-- 


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[hjl dot tools at gmail dot com]

------- Comment #3 from hjl dot tools at gmail dot com  2010-09-17 16:30 -------
For some reason, it only fails with 32bit gcc.


-- 


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[matz at gcc dot gnu dot org]

------- Comment #4 from matz at gcc dot gnu dot org  2010-09-20 11:07 -------
Whoops.  Yeah, I only added x86_64-*-* to the vect_perm targets.  Obviously,
as sse2 is active by default for the vectorizer testsuite I also need to
add the i?86-*-* targets.  H.J., can you try with this patch on a native system
(so that we may see any possible fallout)?

Index: testsuite/lib/target-supports.exp
===================================================================
--- testsuite/lib/target-supports.exp   (revision 164367)
+++ testsuite/lib/target-supports.exp   (working copy)
@@ -2426,6 +2426,7 @@ proc check_effective_target_vect_perm {
         set et_vect_perm_saved 0
         if { [istarget powerpc*-*-*]
              || [istarget spu-*-*]
+            || [istarget i?86-*-*]
             || [istarget x86_64-*-*] } {
             set et_vect_perm_saved 1
         }


-- 


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[hjl dot tools at gmail dot com]

------- Comment #5 from hjl dot tools at gmail dot com  2010-09-20 13:54 -------
(In reply to comment #4)
> Whoops.  Yeah, I only added x86_64-*-* to the vect_perm targets.  Obviously,
> as sse2 is active by default for the vectorizer testsuite I also need to
> add the i?86-*-* targets.  H.J., can you try with this patch on a native system
> (so that we may see any possible fallout)?
> 
> Index: testsuite/lib/target-supports.exp
> ===================================================================
> --- testsuite/lib/target-supports.exp   (revision 164367)
> +++ testsuite/lib/target-supports.exp   (working copy)
> @@ -2426,6 +2426,7 @@ proc check_effective_target_vect_perm {
>          set et_vect_perm_saved 0
>          if { [istarget powerpc*-*-*]
>               || [istarget spu-*-*]
> +            || [istarget i?86-*-*]
>              || [istarget x86_64-*-*] } {
>              set et_vect_perm_saved 1
>          }
> 

It works.


-- 


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

[Bug middle-end/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[matz at gcc dot gnu dot org]

------- Comment #6 from matz at gcc dot gnu dot org  2010-09-20 14:12 -------
Subject: Bug 45706

Author: matz
Date: Mon Sep 20 14:12:04 2010
New Revision: 164433

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164433
Log:
        PR testsuite/45706
        * lib/target-supports.exp (check_effective_target_vect_perm):
        Add i?86-*-*.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/lib/target-supports.exp


-- 


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

[Bug testsuite/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[matz at gcc dot gnu dot org]

------- Comment #7 from matz at gcc dot gnu dot org  2010-09-20 14:14 -------
Fixed.


-- 

matz at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
          Component|middle-end                  |testsuite
         Resolution|                            |FIXED


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

[Bug testsuite/45706] [4.6 regression] gcc.dg/vect/vect-114.c

[matz at gcc dot gnu dot org]

------- Comment #8 from matz at gcc dot gnu dot org  2010-09-20 14:45 -------
Subject: Bug 45706

Author: matz
Date: Mon Sep 20 14:45:30 2010
New Revision: 164435

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164435
Log:
        PR testsuite/45706
        * gcc.dg/vect/pr43432.c: Don't override dg-options, defaults are
        enough.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/vect/pr43432.c


-- 


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