Hi,

In my testing with cost tweaking for vector with length, I found
two cases below didn't get the expected output.  Since the expected
instructions reply on the vectorization occurrence, we don't expect
vectorization gets disabled by cost model.

To make it not fragile, the fix is to force it without vect cost model.

Is it ok for trunk?

BR,
Kewen
---
gcc/testsuite/ChangeLog:

        * gcc.target/powerpc/conv-vectorize-1.c: Add option
        -fno-vect-cost-model.
        * gcc.target/powerpc/conv-vectorize-2.c: Likewise.

------
diff --git a/gcc/testsuite/gcc.target/powerpc/conv-vectorize-1.c 
b/gcc/testsuite/gcc.target/powerpc/conv-vectorize-1.c
index d96db146864..8b8c88befd5 100644
--- a/gcc/testsuite/gcc.target/powerpc/conv-vectorize-1.c
+++ b/gcc/testsuite/gcc.target/powerpc/conv-vectorize-1.c
@@ -1,5 +1,5 @@
 /* { dg-require-effective-target powerpc_vsx_ok } */
-/* { dg-options "-O2 -ftree-vectorize -mvsx" } */
+/* { dg-options "-O2 -ftree-vectorize -mvsx -fno-vect-cost-model" } */

 /* Test vectorizer can exploit vector conversion instructions to convert
    unsigned/signed long long to float.  */
diff --git a/gcc/testsuite/gcc.target/powerpc/conv-vectorize-2.c 
b/gcc/testsuite/gcc.target/powerpc/conv-vectorize-2.c
index 5dd5deabdd3..06ae4e2d5ce 100644
--- a/gcc/testsuite/gcc.target/powerpc/conv-vectorize-2.c
+++ b/gcc/testsuite/gcc.target/powerpc/conv-vectorize-2.c
@@ -1,5 +1,5 @@
 /* { dg-require-effective-target powerpc_vsx_ok } */
-/* { dg-options "-O2 -ftree-vectorize -mvsx" } */
+/* { dg-options "-O2 -ftree-vectorize -mvsx -fno-vect-cost-model" } */

 /* Test vectorizer can exploit vector conversion instructions to convert
    float to unsigned/signed long long.  */

Reply via email to