Bug 45706 - [4.6 regression] gcc.dg/vect/vect-114.c
Summary: [4.6 regression] gcc.dg/vect/vect-114.c
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: testsuite (show other bugs)
Version: 4.6.0
: P3 normal
Target Milestone: 4.6.0
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-17 15:59 UTC by H.J. Lu
Modified: 2010-09-20 14:14 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description H.J. Lu 2010-09-17 15:59:22 UTC 
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
Comment 1 Michael Matz 2010-09-17 16:12:57 UTC 
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.
Comment 2 H.J. Lu 2010-09-17 16:30:14 UTC 
(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
                                                        ^^^^^^^^^^^^^^^^^^^
Comment 3 H.J. Lu 2010-09-17 16:30:46 UTC 
For some reason, it only fails with 32bit gcc.
Comment 4 Michael Matz 2010-09-20 11:07:58 UTC 
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
         }
Comment 5 H.J. Lu 2010-09-20 13:54:40 UTC 
(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.
Comment 6 Michael Matz 2010-09-20 14:12:24 UTC 
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
Comment 7 Michael Matz 2010-09-20 14:14:48 UTC 
Fixed.
Comment 8 Michael Matz 2010-09-20 14:45:54 UTC 
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