diff mbox series

[testsuite,sparc] Don't xfail gcc.dg/vect/vect-multitypes-12.c on 32-bit SPARC (PR tree-optimization/80996)

Message ID yddk211sm46.fsf@CeBiTec.Uni-Bielefeld.DE
State New
Headers show
Series [testsuite,sparc] Don't xfail gcc.dg/vect/vect-multitypes-12.c on 32-bit SPARC (PR tree-optimization/80996) | expand

Commit Message

Rainer Orth Sept. 14, 2017, 8:47 a.m. UTC
Since

2017-06-02  Richard Biener  <rguenther@suse.de>

        * tree-vect-loop.c (vect_analyze_loop_operations): Not relevant
        PHIs are ok.
        * tree-vect-stmts.c (process_use): Do not mark backedge defs
        for inductions as relevant.

gcc.dg/vect/vect-multitypes-12.c XPASSes on 32-bit SPARC:

XPASS: gcc.dg/vect/vect-multitypes-12.c -flto -ffat-lto-objects  scan-tree-dump
-times vect "vectorized 1 loops" 1
XPASS: gcc.dg/vect/vect-multitypes-12.c scan-tree-dump-times vect "vectorized 1
 loops" 1

Fixed by removing the xfail.  Tested with the appropriate runtest
invocation on sparc-sun-solaris2.11, i386-pc-solaris2.11, and
x86_64-pc-linux-gnu.  Ok for mainline?

	Rainer

Comments

Richard Biener Sept. 14, 2017, 9:03 a.m. UTC | #1
On Thu, 14 Sep 2017, Rainer Orth wrote:

> Since
> 
> 2017-06-02  Richard Biener  <rguenther@suse.de>
> 
>         * tree-vect-loop.c (vect_analyze_loop_operations): Not relevant
>         PHIs are ok.
>         * tree-vect-stmts.c (process_use): Do not mark backedge defs
>         for inductions as relevant.
> 
> gcc.dg/vect/vect-multitypes-12.c XPASSes on 32-bit SPARC:
> 
> XPASS: gcc.dg/vect/vect-multitypes-12.c -flto -ffat-lto-objects  scan-tree-dump
> -times vect "vectorized 1 loops" 1
> XPASS: gcc.dg/vect/vect-multitypes-12.c scan-tree-dump-times vect "vectorized 1
>  loops" 1
> 
> Fixed by removing the xfail.  Tested with the appropriate runtest
> invocation on sparc-sun-solaris2.11, i386-pc-solaris2.11, and
> x86_64-pc-linux-gnu.  Ok for mainline?

-/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
target
sparc*-*-* xfail ilp32 } } } */
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
target
sparc*-*-* } } } */
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
target
{ { ! sparc*-*-* }  && { ! vect_unpack } } } } } */

merge the sparc line with the last?  That is, remove the { ! sparc*-*-* }
there?

Ok with that change.

Richard.
Rainer Orth Sept. 14, 2017, 9:22 a.m. UTC | #2
Hi Richard,

> -/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
> target
> sparc*-*-* xfail ilp32 } } } */
> +/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
> target
> sparc*-*-* } } } */
>  /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { 
> target
> { { ! sparc*-*-* }  && { ! vect_unpack } } } } } */
>
> merge the sparc line with the last?  That is, remove the { ! sparc*-*-* }
> there?

of course, thanks for noticing.  Installed like this.

	Rainer
diff mbox series

Patch

# HG changeset patch
# Parent  b1b31b27e4684d67a402b5172d464e79b8ed3fb6
Don't xfail gcc.dg/vect/vect-multitypes-12.c on 32-bit SPARC (PR tree-optimization/80996)

diff --git a/gcc/testsuite/gcc.dg/vect/vect-multitypes-12.c b/gcc/testsuite/gcc.dg/vect/vect-multitypes-12.c
--- a/gcc/testsuite/gcc.dg/vect/vect-multitypes-12.c
+++ b/gcc/testsuite/gcc.dg/vect/vect-multitypes-12.c
@@ -40,6 +40,6 @@  int main (void)
 
 /* bleah */
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 2 "vect" { target vect_unpack } } } */
-/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { target sparc*-*-* xfail ilp32 } } } */
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { target sparc*-*-* } } } */
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { target { { ! sparc*-*-* }  && { ! vect_unpack } } } } } */