]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
testsuite: tree-ssa: Fix i686/-m32 fails for vector-*.c tests
authorChristoph Müllner <christoph.muellner@vrull.eu>
Fri, 20 Dec 2024 15:06:07 +0000 (16:06 +0100)
committerChristoph Müllner <christoph.muellner@vrull.eu>
Fri, 20 Dec 2024 20:05:05 +0000 (21:05 +0100)
FAILs have been reported for several tree-ssa vector-*.c tests
on i686-linux or on x86_64-linux with -m32.
This patch addresses these fails by setting the necessary -msse2 flags.

This patch also streamlines all tests to use dg-options instead
of dg-additional-options.  This is in line with most other tests
in gcc.dg/tree-ssa.

Tested with the following board config in RUNTESTFLAGS:
  --target_board=unix\{-m64,-m32,-m32/-mno-mmx/-mno-sse}

gcc/testsuite/ChangeLog:

* gcc.dg/tree-ssa/satd-hadamard.c: Rename dg-additional-options
to dg-options.
* gcc.dg/tree-ssa/vector-10.c: Rename dg-additional-options
to dg-options and add -msse2 to it.
* gcc.dg/tree-ssa/vector-11.c: Likewise.
* gcc.dg/tree-ssa/vector-8.c: Rename dg-additional-options
to dg-options.
* gcc.dg/tree-ssa/vector-9.c: Likewise.

Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
gcc/testsuite/gcc.dg/tree-ssa/satd-hadamard.c
gcc/testsuite/gcc.dg/tree-ssa/vector-10.c
gcc/testsuite/gcc.dg/tree-ssa/vector-11.c
gcc/testsuite/gcc.dg/tree-ssa/vector-8.c
gcc/testsuite/gcc.dg/tree-ssa/vector-9.c

index 6042378f16507245820e06f721222e0c406f4978..b0d4f4c4a831e294d67365733bec976d278c2b45 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do compile } */
-/* { dg-additional-options "-O3 -fdump-tree-forwprop4-details" } */
+/* { dg-options "-O3 -fdump-tree-forwprop4-details" } */
 /* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
 
 #include <stdint.h>
index bb1ed92dc90af9d18b1ecbb73549e13868191857..4a095a1b54ae3405a3799d86acb543bfd5bad8ef 100644 (file)
@@ -1,5 +1,6 @@
 /* { dg-do compile } */
-/* { dg-additional-options "-O3 -fdump-tree-forwprop1-details -Wno-psabi" } */
+/* { dg-options "-O3 -fdump-tree-forwprop1-details -Wno-psabi" } */
+/* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
 
 typedef int vec __attribute__((vector_size (4 * sizeof (int))));
 
@@ -105,7 +106,7 @@ void f4 (vec *p_v_in_1, vec *p_v_out_1, vec *p_v_out_2)
   v_y = v_1 - v_2;
   v_out_1 = __builtin_shuffle (v_x, v_y, sel);
 
-  /* Won't merge because of dependency.  */
+  /* Won't blend because of dependency.  */
   v_in_2 = foo (v_out_1);
 
   /* Second vec perm sequence.  */
index e4102d318d29b29ac7a23a17e4a702adc131fc04..b1f0c74152a433b2ee65ba6d081dd2517a867760 100644 (file)
@@ -1,5 +1,6 @@
 /* { dg-do compile } */
-/* { dg-additional-options "-O3 -fdump-tree-forwprop1-details -Wno-psabi" } */
+/* { dg-options "-O3 -fdump-tree-forwprop1-details -Wno-psabi" } */
+/* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
 
 typedef int vec __attribute__((vector_size (4 * sizeof (int))));
 
@@ -27,9 +28,6 @@ void f1 (vec *p_v_in, vec *p_v_out_1, vec *p_v_out_2)
   v_y = v_1 + v_2;
   v_out_2 = __builtin_shuffle (v_y, v_x, sel);
 
-  /* Won't blend because the narrowed sequence
-     utilizes three of the four lanes.  */
-
   *p_v_out_1 = v_out_1;
   *p_v_out_2 = v_out_2;
 }
index ba9a0187c106357a21bcba31a822bc93b263780c..b7e0a529c288feb7ecfe9059940edb8bf20c21b3 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do compile } */
-/* { dg-additional-options "-O3 -fdump-tree-forwprop1-details" } */
+/* { dg-options "-O3 -fdump-tree-forwprop1-details" } */
 /* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
 
 typedef int vec __attribute__((vector_size (4 * sizeof (int))));
index 1aa2ef99c3c293181d0263b3b9d6a351a83af2e6..2fb2be6e2a55735d1b5effb8d8d932e7d6cff830 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do compile } */
-/* { dg-additional-options "-O3 -fdump-tree-forwprop1-details" } */
+/* { dg-options "-O3 -fdump-tree-forwprop1-details" } */
 /* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
 
 typedef int vec __attribute__((vector_size (4 * sizeof (int))));