diff mbox series

testsuite: Fix gcc.dg/vect/bb-slp-39.c on IBM Z

Message ID 20210420081727.2816426-1-stefansf@linux.ibm.com
State New
Headers show
Series testsuite: Fix gcc.dg/vect/bb-slp-39.c on IBM Z | expand

Commit Message

Stefan Schulze Frielinghaus April 20, 2021, 8:17 a.m. UTC 
On IBM Z the aliasing stores are realized through one element vector
instructions, if no cost model for vectorization is used which is the
default according to vect.exp.  Fixed by changing the number of times
the pattern must be found in the dump.

gcc/testsuite/ChangeLog:

	* gcc.dg/vect/bb-slp-39.c: Change number of times the pattern
	must match for target IBM Z only.

Ok for mainline?

---
 gcc/testsuite/gcc.dg/vect/bb-slp-39.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Jeff Law April 20, 2021, 11:07 p.m. UTC | #1 
On 4/20/2021 2:17 AM, Stefan Schulze Frielinghaus via Gcc-patches wrote:
> On IBM Z the aliasing stores are realized through one element vector
> instructions, if no cost model for vectorization is used which is the
> default according to vect.exp.  Fixed by changing the number of times
> the pattern must be found in the dump.
>
> gcc/testsuite/ChangeLog:
>
> 	* gcc.dg/vect/bb-slp-39.c: Change number of times the pattern
> 	must match for target IBM Z only.

OK

jeff
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.dg/vect/bb-slp-39.c b/gcc/testsuite/gcc.dg/vect/bb-slp-39.c
index 255bb1095dc..ee596cfa08b 100644
--- a/gcc/testsuite/gcc.dg/vect/bb-slp-39.c
+++ b/gcc/testsuite/gcc.dg/vect/bb-slp-39.c
@@ -16,4 +16,5 @@  void foo (double *p)
 }
 
 /* See that we vectorize three SLP instances.  */
-/* { dg-final { scan-tree-dump-times "vectorizing stmts using SLP" 3 "slp2" } } */
+/* { dg-final { scan-tree-dump-times "vectorizing stmts using SLP" 3 "slp2" { target { ! s390*-*-* } } } } */
+/* { dg-final { scan-tree-dump-times "vectorizing stmts using SLP" 5 "slp2" { target {   s390*-*-* } } } } */