diff mbox series

testsuite: Fix copy-headers-8.c

Message ID 20240326084204.1147691-2-stefansf@linux.ibm.com
State New
Headers show
Series testsuite: Fix copy-headers-8.c | expand

Commit Message

Stefan Schulze Frielinghaus March 26, 2024, 8:42 a.m. UTC 
This fixes the test on s390x.  I'm also seeing test failures for
riscv64-suse-linux-gnu, m68k-unknown-linux-gnu, pru-unknown-elf, and
powerpc64le-unknown-linux-gnu.  However, I didn't check them so this
might or might not fix those, too.

OK for mainline?

gcc/testsuite/ChangeLog:

	* gcc.dg/tree-ssa/copy-headers-8.c: Set
	LOGICAL_OP_NON_SHORT_CIRCUIT to true.
---
 gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Richard Biener March 26, 2024, 9:18 a.m. UTC | #1 
On Tue, Mar 26, 2024 at 9:43 AM Stefan Schulze Frielinghaus
<stefansf@linux.ibm.com> wrote:
>
> This fixes the test on s390x.  I'm also seeing test failures for
> riscv64-suse-linux-gnu, m68k-unknown-linux-gnu, pru-unknown-elf, and
> powerpc64le-unknown-linux-gnu.  However, I didn't check them so this
> might or might not fix those, too.
>
> OK for mainline?

OK.

> gcc/testsuite/ChangeLog:
>
>         * gcc.dg/tree-ssa/copy-headers-8.c: Set
>         LOGICAL_OP_NON_SHORT_CIRCUIT to true.
> ---
>  gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c b/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
> index 8b4b5e7ea81..28b4d15d87f 100644
> --- a/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
> +++ b/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
> @@ -1,5 +1,8 @@
> +/* For targets where LOGICAL_OP_NON_SHORT_CIRCUIT evaluates to false, two
> +   conditional jumps are emitted instead of a combined conditional which this
> +   test is all about.  Thus, set it to true.  */
>  /* { dg-do compile } */
> -/* { dg-options "-O2 -fdump-tree-ch2-details" } */
> +/* { dg-options "-O2 -fdump-tree-ch2-details --param logical-op-non-short-circuit=1" } */
>
>  int is_sorted(int *a, int n, int m, int k)
>  {
> --
> 2.43.0
>
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c b/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
index 8b4b5e7ea81..28b4d15d87f 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/copy-headers-8.c
@@ -1,5 +1,8 @@ 
+/* For targets where LOGICAL_OP_NON_SHORT_CIRCUIT evaluates to false, two
+   conditional jumps are emitted instead of a combined conditional which this
+   test is all about.  Thus, set it to true.  */
 /* { dg-do compile } */
-/* { dg-options "-O2 -fdump-tree-ch2-details" } */
+/* { dg-options "-O2 -fdump-tree-ch2-details --param logical-op-non-short-circuit=1" } */
 
 int is_sorted(int *a, int n, int m, int k)
 {