diff mbox series

MIPS: Adjust mips16e2 related tests for ifcvt costing changes

Message ID 20230704095048.1065139-1-jie.mei@oss.cipunited.com
State Unresolved
Headers
Series MIPS: Adjust mips16e2 related tests for ifcvt costing changes |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

梅杰 July 4, 2023, 9:50 a.m. UTC 
  A mips16e2 related test fails after the ifcvt change. The mips16e2
addition also causes a test for unrelated module to fail.

This patch adjusts branch costs when running the two affected tests.

These tests should not require the -mbranch-cost option, and
this issue needs to be addressed.

gcc/testsuite/ChangeLog:

        * gcc.target/mips/mips16e2-cmov.c: Adjust branch cost to
        encourage if-conversion.
        * gcc.target/mips/movcc-3.c: Same as above.
---
 gcc/testsuite/gcc.target/mips/mips16e2-cmov.c | 2 +-
 gcc/testsuite/gcc.target/mips/movcc-3.c       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

YunQiang Su July 5, 2023, 1:36 a.m. UTC | #1 
Jie Mei <jie.mei@oss.cipunited.com> 于2023年7月4日周二 17:52写道：
>
> A mips16e2 related test fails after the ifcvt change. The mips16e2
> addition also causes a test for unrelated module to fail.
>
> This patch adjusts branch costs when running the two affected tests.
>
> These tests should not require the -mbranch-cost option, and
> this issue needs to be addressed.
>

Wish it soon.

> gcc/testsuite/ChangeLog:
>
>         * gcc.target/mips/mips16e2-cmov.c: Adjust branch cost to
>         encourage if-conversion.
>         * gcc.target/mips/movcc-3.c: Same as above.
> ---
>  gcc/testsuite/gcc.target/mips/mips16e2-cmov.c | 2 +-
>  gcc/testsuite/gcc.target/mips/movcc-3.c       | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c b/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
> index 6e9dd82ebf3..129ea23b65b 100644
> --- a/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
> +++ b/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
> @@ -1,4 +1,4 @@
> -/* { dg-options "-mno-abicalls -mgpopt -G8 -mabi=32 -mips16 -mmips16e2" } */
> +/* { dg-options "-mno-abicalls -mgpopt -G8 -mabi=32 -mips16 -mmips16e2 -mbranch-cost=2" } */
>  /* { dg-skip-if "code quality test" { *-*-* } { "-O0" } { "" } } */
>
>  /* Test MOVN.  */
> diff --git a/gcc/testsuite/gcc.target/mips/movcc-3.c b/gcc/testsuite/gcc.target/mips/movcc-3.c
> index 80d44098a3f..569a00423c1 100644
> --- a/gcc/testsuite/gcc.target/mips/movcc-3.c
> +++ b/gcc/testsuite/gcc.target/mips/movcc-3.c
> @@ -1,5 +1,5 @@
>  /* { dg-do compile } */
> -/* { dg-options "(HAS_MOVN) -mhard-float -mbranch-cost=2" } */
> +/* { dg-options "(HAS_MOVN) -mhard-float -mbranch-cost=3" } */
>  /* { dg-skip-if "code quality test" { *-*-* } { "-O0" "-Os" } { "" } } */
>  /* { dg-final { scan-assembler "\tmovt\t" } } */
>  /* { dg-final { scan-assembler "\tmovf\t" } } */
> --
> 2.40.1
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c b/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
index 6e9dd82ebf3..129ea23b65b 100644
--- a/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
+++ b/gcc/testsuite/gcc.target/mips/mips16e2-cmov.c
@@ -1,4 +1,4 @@ 
-/* { dg-options "-mno-abicalls -mgpopt -G8 -mabi=32 -mips16 -mmips16e2" } */
+/* { dg-options "-mno-abicalls -mgpopt -G8 -mabi=32 -mips16 -mmips16e2 -mbranch-cost=2" } */
 /* { dg-skip-if "code quality test" { *-*-* } { "-O0" } { "" } } */
 
 /* Test MOVN.  */
diff --git a/gcc/testsuite/gcc.target/mips/movcc-3.c b/gcc/testsuite/gcc.target/mips/movcc-3.c
index 80d44098a3f..569a00423c1 100644
--- a/gcc/testsuite/gcc.target/mips/movcc-3.c
+++ b/gcc/testsuite/gcc.target/mips/movcc-3.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-options "(HAS_MOVN) -mhard-float -mbranch-cost=2" } */
+/* { dg-options "(HAS_MOVN) -mhard-float -mbranch-cost=3" } */
 /* { dg-skip-if "code quality test" { *-*-* } { "-O0" "-Os" } { "" } } */
 /* { dg-final { scan-assembler "\tmovt\t" } } */
 /* { dg-final { scan-assembler "\tmovf\t" } } */