public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] RISC-V/testsuite: constraint some of tests to hard_float
@ 2022-05-27  1:44 Vineet Gupta
  2022-05-27  1:56 ` [PATCH v2] " Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2022-05-27  1:44 UTC (permalink / raw)
  To: gcc-patches
  Cc: kito.cheng, Philipp Tomsich, Palmer Dabbelt,
	Christoph Müllner, gnu-toolchain, Andrew Waterman,
	Maciej W . Rozycki, Vineet Gupta, Vineet Gupta

From: Vineet Gupta <vgupta@kernel.org>

Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
tests which check for hard float instructions which obviously fails on
soft-float ABI builds.

So constraint with "dg-require-effective-target hard_float"

This reduces bunch of new RV failures on trunk since that commit.

|               ========= Summary of gcc testsuite =========
|                            | # of unexpected case / # of unique unexpected case
|                            |          gcc |          g++ |     gfortran |
|   rv64imac/   lp64/ medlow |  134 /    22 |    0 /     0 |            - |  BEFORE
|   rv64imac/   lp64/ medlow |   30 /    10 |    0 /     0 |            - |  AFTER
|

gcc/testsuite/Changelog:
	* gcc.target/riscv/fmax.c: Add dg-require-effective-target hard_float.
	* gcc.target/riscv/fmaxf.c: Ditto.
	* gcc.target/riscv/fmin.c: Ditto.
	* gcc.target/riscv/fminf.c: Ditto.
	* gcc.target/riscv/smax-ieee.c: Ditto.
	* gcc.target/riscv/smax.c: Ditto.
	* gcc.target/riscv/smaxf-ieee.c: Ditto.
	* gcc.target/riscv/smaxf.c: Ditto.
	* gcc.target/riscv/smin-ieee.c: Ditto.
	* gcc.target/riscv/smin.c: Ditto.
	* gcc.target/riscv/sminf-ieee.c: Ditto.
	* gcc.target/riscv/sminf.c: Ditto.

Signed-off-by: Vineet Gupta <vgupta@kernel.org>
Signed-off-by: Vineet Gupta <vineetg@rivosinc.com>
---
 gcc/testsuite/gcc.target/riscv/fmax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fmaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/fmin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fminf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/smax-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/smin-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/sminf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/sminf.c      | 1 +
 12 files changed, 12 insertions(+)

diff --git a/gcc/testsuite/gcc.target/riscv/fmax.c b/gcc/testsuite/gcc.target/riscv/fmax.c
index c71d35c9f9dc..e1b7fa8f918c 100644
--- a/gcc/testsuite/gcc.target/riscv/fmax.c
+++ b/gcc/testsuite/gcc.target/riscv/fmax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fmaxf.c b/gcc/testsuite/gcc.target/riscv/fmaxf.c
index f9980166887a..8da0513dc8f6 100644
--- a/gcc/testsuite/gcc.target/riscv/fmaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/fmaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/fmin.c b/gcc/testsuite/gcc.target/riscv/fmin.c
index 9634abd19af8..01993d49bc21 100644
--- a/gcc/testsuite/gcc.target/riscv/fmin.c
+++ b/gcc/testsuite/gcc.target/riscv/fmin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fminf.c b/gcc/testsuite/gcc.target/riscv/fminf.c
index 9a3687be3092..32ce363e10d8 100644
--- a/gcc/testsuite/gcc.target/riscv/fminf.c
+++ b/gcc/testsuite/gcc.target/riscv/fminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smax-ieee.c b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
index 3a98aeb45add..2dbccefe2f4d 100644
--- a/gcc/testsuite/gcc.target/riscv/smax-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smax.c b/gcc/testsuite/gcc.target/riscv/smax.c
index d806c632faed..1092cc49f25e 100644
--- a/gcc/testsuite/gcc.target/riscv/smax.c
+++ b/gcc/testsuite/gcc.target/riscv/smax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
index 6cf23d789b6e..31b9bfa4d91e 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf.c b/gcc/testsuite/gcc.target/riscv/smaxf.c
index d6a7a7f84cf7..aa1f22b45a17 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smin-ieee.c b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
index c0a148c37cf1..ea36c2de89fe 100644
--- a/gcc/testsuite/gcc.target/riscv/smin-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smin.c b/gcc/testsuite/gcc.target/riscv/smin.c
index e325e9a1c7cc..d07a0fdc1525 100644
--- a/gcc/testsuite/gcc.target/riscv/smin.c
+++ b/gcc/testsuite/gcc.target/riscv/smin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
index 353e7a18704a..427617af08f1 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/sminf.c b/gcc/testsuite/gcc.target/riscv/sminf.c
index f0ba7b43bdf9..7df223e8bcad 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
-- 
2.32.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v2] RISC-V/testsuite: constraint some of tests to hard_float
  2022-05-27  1:44 [PATCH] RISC-V/testsuite: constraint some of tests to hard_float Vineet Gupta
@ 2022-05-27  1:56 ` Vineet Gupta
  2022-05-27  2:36   ` [PATCH v3] " Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2022-05-27  1:56 UTC (permalink / raw)
  To: gcc-patches
  Cc: kito.cheng, Philipp Tomsich, Palmer Dabbelt,
	Christoph Müllner, gnu-toolchain, Andrew Waterman,
	Maciej W . Rozycki, Vineet Gupta

Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
tests which check for hard float instructions which obviously fails on
soft-float ABI builds.

So constraint with "dg-require-effective-target hard_float"

This reduces bunch of new RV failures on trunk since that commit.

|               ========= Summary of gcc testsuite =========
|                            | # of unexpected case / # of unique unexpected case
|                            |          gcc |          g++ |     gfortran |
|   rv64imac/   lp64/ medlow |  134 /    22 |    0 /     0 |            - |  BEFORE
|   rv64imac/   lp64/ medlow |   30 /    10 |    0 /     0 |            - |  AFTER
|

gcc/testsuite/Changelog:
	* gcc.target/riscv/fmax.c: Add dg-require-effective-target hard_float.
	* gcc.target/riscv/fmaxf.c: Ditto.
	* gcc.target/riscv/fmin.c: Ditto.
	* gcc.target/riscv/fminf.c: Ditto.
	* gcc.target/riscv/smax-ieee.c: Ditto.
	* gcc.target/riscv/smax.c: Ditto.
	* gcc.target/riscv/smaxf-ieee.c: Ditto.
	* gcc.target/riscv/smaxf.c: Ditto.
	* gcc.target/riscv/smin-ieee.c: Ditto.
	* gcc.target/riscv/smin.c: Ditto.
	* gcc.target/riscv/sminf-ieee.c: Ditto.
	* gcc.target/riscv/sminf.c: Ditto.

Signed-off-by: Vineet Gupta <vineetg@rivosinc.com>
---
v2:
    Fixed the SoB snafu in v1
---
 gcc/testsuite/gcc.target/riscv/fmax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fmaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/fmin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fminf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/smax-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/smin-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/sminf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/sminf.c      | 1 +
 12 files changed, 12 insertions(+)

diff --git a/gcc/testsuite/gcc.target/riscv/fmax.c b/gcc/testsuite/gcc.target/riscv/fmax.c
index c71d35c9f9dc..e1b7fa8f918c 100644
--- a/gcc/testsuite/gcc.target/riscv/fmax.c
+++ b/gcc/testsuite/gcc.target/riscv/fmax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fmaxf.c b/gcc/testsuite/gcc.target/riscv/fmaxf.c
index f9980166887a..8da0513dc8f6 100644
--- a/gcc/testsuite/gcc.target/riscv/fmaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/fmaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/fmin.c b/gcc/testsuite/gcc.target/riscv/fmin.c
index 9634abd19af8..01993d49bc21 100644
--- a/gcc/testsuite/gcc.target/riscv/fmin.c
+++ b/gcc/testsuite/gcc.target/riscv/fmin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fminf.c b/gcc/testsuite/gcc.target/riscv/fminf.c
index 9a3687be3092..32ce363e10d8 100644
--- a/gcc/testsuite/gcc.target/riscv/fminf.c
+++ b/gcc/testsuite/gcc.target/riscv/fminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smax-ieee.c b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
index 3a98aeb45add..2dbccefe2f4d 100644
--- a/gcc/testsuite/gcc.target/riscv/smax-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smax.c b/gcc/testsuite/gcc.target/riscv/smax.c
index d806c632faed..1092cc49f25e 100644
--- a/gcc/testsuite/gcc.target/riscv/smax.c
+++ b/gcc/testsuite/gcc.target/riscv/smax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
index 6cf23d789b6e..31b9bfa4d91e 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf.c b/gcc/testsuite/gcc.target/riscv/smaxf.c
index d6a7a7f84cf7..aa1f22b45a17 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smin-ieee.c b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
index c0a148c37cf1..ea36c2de89fe 100644
--- a/gcc/testsuite/gcc.target/riscv/smin-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smin.c b/gcc/testsuite/gcc.target/riscv/smin.c
index e325e9a1c7cc..d07a0fdc1525 100644
--- a/gcc/testsuite/gcc.target/riscv/smin.c
+++ b/gcc/testsuite/gcc.target/riscv/smin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
index 353e7a18704a..427617af08f1 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/sminf.c b/gcc/testsuite/gcc.target/riscv/sminf.c
index f0ba7b43bdf9..7df223e8bcad 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
-- 
2.32.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3] RISC-V/testsuite: constraint some of tests to hard_float
  2022-05-27  1:56 ` [PATCH v2] " Vineet Gupta
@ 2022-05-27  2:36   ` Vineet Gupta
  2022-05-30  3:50     ` Kito Cheng
  2022-06-01 11:51     ` Maciej W. Rozycki
  0 siblings, 2 replies; 7+ messages in thread
From: Vineet Gupta @ 2022-05-27  2:36 UTC (permalink / raw)
  To: gcc-patches
  Cc: kito.cheng, Philipp Tomsich, Palmer Dabbelt,
	Christoph Müllner, gnu-toolchain, Andrew Waterman,
	Maciej W . Rozycki, Vineet Gupta

Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
tests which check for hard float instructions which obviously fails on
soft-float ABI builds.

And my recent commit b646d7d279ae ("RISC-V: Inhibit FP <--> int register
moves via tune param") is guilty of same crime.

So constraint with "dg-require-effective-target hard_float"

This reduces bunch of new RV failures.

|               ========= Summary of gcc testsuite =========
|                            | # of unexpected case / # of unique unexpected case
|                            |          gcc |          g++ |     gfortran |
|   rv64imac/   lp64/ medlow |  134 /    22 |    0 /     0 |            - |  BEFORE
|   rv64imac/   lp64/ medlow |   22 /     9 |    0 /     0 |            - |  AFTER
|

gcc/testsuite/Changelog:
	* gcc.target/riscv/fmax.c: Add dg-require-effective-target hard_float.
	* gcc.target/riscv/fmaxf.c: Ditto.
	* gcc.target/riscv/fmin.c: Ditto.
	* gcc.target/riscv/fminf.c: Ditto.
	* gcc.target/riscv/smax-ieee.c: Ditto.
	* gcc.target/riscv/smax.c: Ditto.
	* gcc.target/riscv/smaxf-ieee.c: Ditto.
	* gcc.target/riscv/smaxf.c: Ditto.
	* gcc.target/riscv/smin-ieee.c: Ditto.
	* gcc.target/riscv/smin.c: Ditto.
	* gcc.target/riscv/sminf-ieee.c: Ditto.
	* gcc.target/riscv/sminf.c: Ditto.
	* gcc.target/riscv/pr105666.c: Ditto.

Signed-off-by: Vineet Gupta <vineetg@rivosinc.com>
---
v3:
    Added fix to pr105666.c as well.
v2:
    Fixed the SoB snafu in v1
---
 gcc/testsuite/gcc.target/riscv/fmax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fmaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/fmin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/fminf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/pr105666.c   | 1 +
 gcc/testsuite/gcc.target/riscv/smax-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smax.c       | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/smaxf.c      | 1 +
 gcc/testsuite/gcc.target/riscv/smin-ieee.c  | 1 +
 gcc/testsuite/gcc.target/riscv/smin.c       | 1 +
 gcc/testsuite/gcc.target/riscv/sminf-ieee.c | 1 +
 gcc/testsuite/gcc.target/riscv/sminf.c      | 1 +
 13 files changed, 13 insertions(+)

diff --git a/gcc/testsuite/gcc.target/riscv/fmax.c b/gcc/testsuite/gcc.target/riscv/fmax.c
index c71d35c9f9dc..e1b7fa8f918c 100644
--- a/gcc/testsuite/gcc.target/riscv/fmax.c
+++ b/gcc/testsuite/gcc.target/riscv/fmax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fmaxf.c b/gcc/testsuite/gcc.target/riscv/fmaxf.c
index f9980166887a..8da0513dc8f6 100644
--- a/gcc/testsuite/gcc.target/riscv/fmaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/fmaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/fmin.c b/gcc/testsuite/gcc.target/riscv/fmin.c
index 9634abd19af8..01993d49bc21 100644
--- a/gcc/testsuite/gcc.target/riscv/fmin.c
+++ b/gcc/testsuite/gcc.target/riscv/fmin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/fminf.c b/gcc/testsuite/gcc.target/riscv/fminf.c
index 9a3687be3092..32ce363e10d8 100644
--- a/gcc/testsuite/gcc.target/riscv/fminf.c
+++ b/gcc/testsuite/gcc.target/riscv/fminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/pr105666.c b/gcc/testsuite/gcc.target/riscv/pr105666.c
index 904f3bc0763f..dd996eec8efc 100644
--- a/gcc/testsuite/gcc.target/riscv/pr105666.c
+++ b/gcc/testsuite/gcc.target/riscv/pr105666.c
@@ -6,6 +6,7 @@
    spilling to stack.  */
 
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-march=rv64g -ffast-math" } */
 
 #define NITER 4
diff --git a/gcc/testsuite/gcc.target/riscv/smax-ieee.c b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
index 3a98aeb45add..2dbccefe2f4d 100644
--- a/gcc/testsuite/gcc.target/riscv/smax-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smax.c b/gcc/testsuite/gcc.target/riscv/smax.c
index d806c632faed..1092cc49f25e 100644
--- a/gcc/testsuite/gcc.target/riscv/smax.c
+++ b/gcc/testsuite/gcc.target/riscv/smax.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
index 6cf23d789b6e..31b9bfa4d91e 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smaxf.c b/gcc/testsuite/gcc.target/riscv/smaxf.c
index d6a7a7f84cf7..aa1f22b45a17 100644
--- a/gcc/testsuite/gcc.target/riscv/smaxf.c
+++ b/gcc/testsuite/gcc.target/riscv/smaxf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/smin-ieee.c b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
index c0a148c37cf1..ea36c2de89fe 100644
--- a/gcc/testsuite/gcc.target/riscv/smin-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/smin.c b/gcc/testsuite/gcc.target/riscv/smin.c
index e325e9a1c7cc..d07a0fdc1525 100644
--- a/gcc/testsuite/gcc.target/riscv/smin.c
+++ b/gcc/testsuite/gcc.target/riscv/smin.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 double
diff --git a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
index 353e7a18704a..427617af08f1 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
 
 float
diff --git a/gcc/testsuite/gcc.target/riscv/sminf.c b/gcc/testsuite/gcc.target/riscv/sminf.c
index f0ba7b43bdf9..7df223e8bcad 100644
--- a/gcc/testsuite/gcc.target/riscv/sminf.c
+++ b/gcc/testsuite/gcc.target/riscv/sminf.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target hard_float } */
 /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
 
 float
-- 
2.32.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v3] RISC-V/testsuite: constraint some of tests to hard_float
  2022-05-27  2:36   ` [PATCH v3] " Vineet Gupta
@ 2022-05-30  3:50     ` Kito Cheng
  2022-07-14  4:29       ` Vineet Gupta
  2022-06-01 11:51     ` Maciej W. Rozycki
  1 sibling, 1 reply; 7+ messages in thread
From: Kito Cheng @ 2022-05-30  3:50 UTC (permalink / raw)
  To: Vineet Gupta
  Cc: GCC Patches, Maciej W . Rozycki, Andrew Waterman,
	Philipp Tomsich, gnu-toolchain

Committed, thanks!

On Fri, May 27, 2022 at 10:37 AM Vineet Gupta <vineetg@rivosinc.com> wrote:
>
> Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
> tests which check for hard float instructions which obviously fails on
> soft-float ABI builds.
>
> And my recent commit b646d7d279ae ("RISC-V: Inhibit FP <--> int register
> moves via tune param") is guilty of same crime.
>
> So constraint with "dg-require-effective-target hard_float"
>
> This reduces bunch of new RV failures.
>
> |               ========= Summary of gcc testsuite =========
> |                            | # of unexpected case / # of unique unexpected case
> |                            |          gcc |          g++ |     gfortran |
> |   rv64imac/   lp64/ medlow |  134 /    22 |    0 /     0 |            - |  BEFORE
> |   rv64imac/   lp64/ medlow |   22 /     9 |    0 /     0 |            - |  AFTER
> |
>
> gcc/testsuite/Changelog:
>         * gcc.target/riscv/fmax.c: Add dg-require-effective-target hard_float.
>         * gcc.target/riscv/fmaxf.c: Ditto.
>         * gcc.target/riscv/fmin.c: Ditto.
>         * gcc.target/riscv/fminf.c: Ditto.
>         * gcc.target/riscv/smax-ieee.c: Ditto.
>         * gcc.target/riscv/smax.c: Ditto.
>         * gcc.target/riscv/smaxf-ieee.c: Ditto.
>         * gcc.target/riscv/smaxf.c: Ditto.
>         * gcc.target/riscv/smin-ieee.c: Ditto.
>         * gcc.target/riscv/smin.c: Ditto.
>         * gcc.target/riscv/sminf-ieee.c: Ditto.
>         * gcc.target/riscv/sminf.c: Ditto.
>         * gcc.target/riscv/pr105666.c: Ditto.
>
> Signed-off-by: Vineet Gupta <vineetg@rivosinc.com>
> ---
> v3:
>     Added fix to pr105666.c as well.
> v2:
>     Fixed the SoB snafu in v1
> ---
>  gcc/testsuite/gcc.target/riscv/fmax.c       | 1 +
>  gcc/testsuite/gcc.target/riscv/fmaxf.c      | 1 +
>  gcc/testsuite/gcc.target/riscv/fmin.c       | 1 +
>  gcc/testsuite/gcc.target/riscv/fminf.c      | 1 +
>  gcc/testsuite/gcc.target/riscv/pr105666.c   | 1 +
>  gcc/testsuite/gcc.target/riscv/smax-ieee.c  | 1 +
>  gcc/testsuite/gcc.target/riscv/smax.c       | 1 +
>  gcc/testsuite/gcc.target/riscv/smaxf-ieee.c | 1 +
>  gcc/testsuite/gcc.target/riscv/smaxf.c      | 1 +
>  gcc/testsuite/gcc.target/riscv/smin-ieee.c  | 1 +
>  gcc/testsuite/gcc.target/riscv/smin.c       | 1 +
>  gcc/testsuite/gcc.target/riscv/sminf-ieee.c | 1 +
>  gcc/testsuite/gcc.target/riscv/sminf.c      | 1 +
>  13 files changed, 13 insertions(+)
>
> diff --git a/gcc/testsuite/gcc.target/riscv/fmax.c b/gcc/testsuite/gcc.target/riscv/fmax.c
> index c71d35c9f9dc..e1b7fa8f918c 100644
> --- a/gcc/testsuite/gcc.target/riscv/fmax.c
> +++ b/gcc/testsuite/gcc.target/riscv/fmax.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/fmaxf.c b/gcc/testsuite/gcc.target/riscv/fmaxf.c
> index f9980166887a..8da0513dc8f6 100644
> --- a/gcc/testsuite/gcc.target/riscv/fmaxf.c
> +++ b/gcc/testsuite/gcc.target/riscv/fmaxf.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>
>  float
> diff --git a/gcc/testsuite/gcc.target/riscv/fmin.c b/gcc/testsuite/gcc.target/riscv/fmin.c
> index 9634abd19af8..01993d49bc21 100644
> --- a/gcc/testsuite/gcc.target/riscv/fmin.c
> +++ b/gcc/testsuite/gcc.target/riscv/fmin.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/fminf.c b/gcc/testsuite/gcc.target/riscv/fminf.c
> index 9a3687be3092..32ce363e10d8 100644
> --- a/gcc/testsuite/gcc.target/riscv/fminf.c
> +++ b/gcc/testsuite/gcc.target/riscv/fminf.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>
>  float
> diff --git a/gcc/testsuite/gcc.target/riscv/pr105666.c b/gcc/testsuite/gcc.target/riscv/pr105666.c
> index 904f3bc0763f..dd996eec8efc 100644
> --- a/gcc/testsuite/gcc.target/riscv/pr105666.c
> +++ b/gcc/testsuite/gcc.target/riscv/pr105666.c
> @@ -6,6 +6,7 @@
>     spilling to stack.  */
>
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-march=rv64g -ffast-math" } */
>
>  #define NITER 4
> diff --git a/gcc/testsuite/gcc.target/riscv/smax-ieee.c b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
> index 3a98aeb45add..2dbccefe2f4d 100644
> --- a/gcc/testsuite/gcc.target/riscv/smax-ieee.c
> +++ b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/smax.c b/gcc/testsuite/gcc.target/riscv/smax.c
> index d806c632faed..1092cc49f25e 100644
> --- a/gcc/testsuite/gcc.target/riscv/smax.c
> +++ b/gcc/testsuite/gcc.target/riscv/smax.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
> index 6cf23d789b6e..31b9bfa4d91e 100644
> --- a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
> +++ b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>
>  float
> diff --git a/gcc/testsuite/gcc.target/riscv/smaxf.c b/gcc/testsuite/gcc.target/riscv/smaxf.c
> index d6a7a7f84cf7..aa1f22b45a17 100644
> --- a/gcc/testsuite/gcc.target/riscv/smaxf.c
> +++ b/gcc/testsuite/gcc.target/riscv/smaxf.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>
>  float
> diff --git a/gcc/testsuite/gcc.target/riscv/smin-ieee.c b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
> index c0a148c37cf1..ea36c2de89fe 100644
> --- a/gcc/testsuite/gcc.target/riscv/smin-ieee.c
> +++ b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/smin.c b/gcc/testsuite/gcc.target/riscv/smin.c
> index e325e9a1c7cc..d07a0fdc1525 100644
> --- a/gcc/testsuite/gcc.target/riscv/smin.c
> +++ b/gcc/testsuite/gcc.target/riscv/smin.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>
>  double
> diff --git a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
> index 353e7a18704a..427617af08f1 100644
> --- a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
> +++ b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>
>  float
> diff --git a/gcc/testsuite/gcc.target/riscv/sminf.c b/gcc/testsuite/gcc.target/riscv/sminf.c
> index f0ba7b43bdf9..7df223e8bcad 100644
> --- a/gcc/testsuite/gcc.target/riscv/sminf.c
> +++ b/gcc/testsuite/gcc.target/riscv/sminf.c
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-require-effective-target hard_float } */
>  /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>
>  float
> --
> 2.32.0
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v3] RISC-V/testsuite: constraint some of tests to hard_float
  2022-05-27  2:36   ` [PATCH v3] " Vineet Gupta
  2022-05-30  3:50     ` Kito Cheng
@ 2022-06-01 11:51     ` Maciej W. Rozycki
  1 sibling, 0 replies; 7+ messages in thread
From: Maciej W. Rozycki @ 2022-06-01 11:51 UTC (permalink / raw)
  To: Vineet Gupta
  Cc: gcc-patches, kito.cheng, Philipp Tomsich, Palmer Dabbelt,
	Christoph Müllner, gnu-toolchain, Andrew Waterman

On Thu, 26 May 2022, Vineet Gupta wrote:

> Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
> tests which check for hard float instructions which obviously fails on
> soft-float ABI builds.

 Sorry to miss it and thank you for the fix!

  Maciej

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v3] RISC-V/testsuite: constraint some of tests to hard_float
  2022-05-30  3:50     ` Kito Cheng
@ 2022-07-14  4:29       ` Vineet Gupta
  2022-07-21  9:23         ` Kito Cheng
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2022-07-14  4:29 UTC (permalink / raw)
  To: Kito Cheng, Vineet Gupta
  Cc: Maciej W . Rozycki, gnu-toolchain, GCC Patches, Andrew Waterman,
	Philipp Tomsich

On 5/29/22 20:50, Kito Cheng via Gcc-patches wrote:
> Committed, thanks!

Can this be backported to gcc-12 please.

Thx,
-Vineet


> On Fri, May 27, 2022 at 10:37 AM Vineet Gupta <vineetg@rivosinc.com> wrote:
>>
>> Commit 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax' RTL pattern") added
>> tests which check for hard float instructions which obviously fails on
>> soft-float ABI builds.
>>
>> And my recent commit b646d7d279ae ("RISC-V: Inhibit FP <--> int register
>> moves via tune param") is guilty of same crime.
>>
>> So constraint with "dg-require-effective-target hard_float"
>>
>> This reduces bunch of new RV failures.
>>
>> |               ========= Summary of gcc testsuite =========
>> |                            | # of unexpected case / # of unique unexpected case
>> |                            |          gcc |          g++ |     gfortran |
>> |   rv64imac/   lp64/ medlow |  134 /    22 |    0 /     0 |            - |  BEFORE
>> |   rv64imac/   lp64/ medlow |   22 /     9 |    0 /     0 |            - |  AFTER
>> |
>>
>> gcc/testsuite/Changelog:
>>          * gcc.target/riscv/fmax.c: Add dg-require-effective-target hard_float.
>>          * gcc.target/riscv/fmaxf.c: Ditto.
>>          * gcc.target/riscv/fmin.c: Ditto.
>>          * gcc.target/riscv/fminf.c: Ditto.
>>          * gcc.target/riscv/smax-ieee.c: Ditto.
>>          * gcc.target/riscv/smax.c: Ditto.
>>          * gcc.target/riscv/smaxf-ieee.c: Ditto.
>>          * gcc.target/riscv/smaxf.c: Ditto.
>>          * gcc.target/riscv/smin-ieee.c: Ditto.
>>          * gcc.target/riscv/smin.c: Ditto.
>>          * gcc.target/riscv/sminf-ieee.c: Ditto.
>>          * gcc.target/riscv/sminf.c: Ditto.
>>          * gcc.target/riscv/pr105666.c: Ditto.
>>
>> Signed-off-by: Vineet Gupta <vineetg@rivosinc.com>
>> ---
>> v3:
>>      Added fix to pr105666.c as well.
>> v2:
>>      Fixed the SoB snafu in v1
>> ---
>>   gcc/testsuite/gcc.target/riscv/fmax.c       | 1 +
>>   gcc/testsuite/gcc.target/riscv/fmaxf.c      | 1 +
>>   gcc/testsuite/gcc.target/riscv/fmin.c       | 1 +
>>   gcc/testsuite/gcc.target/riscv/fminf.c      | 1 +
>>   gcc/testsuite/gcc.target/riscv/pr105666.c   | 1 +
>>   gcc/testsuite/gcc.target/riscv/smax-ieee.c  | 1 +
>>   gcc/testsuite/gcc.target/riscv/smax.c       | 1 +
>>   gcc/testsuite/gcc.target/riscv/smaxf-ieee.c | 1 +
>>   gcc/testsuite/gcc.target/riscv/smaxf.c      | 1 +
>>   gcc/testsuite/gcc.target/riscv/smin-ieee.c  | 1 +
>>   gcc/testsuite/gcc.target/riscv/smin.c       | 1 +
>>   gcc/testsuite/gcc.target/riscv/sminf-ieee.c | 1 +
>>   gcc/testsuite/gcc.target/riscv/sminf.c      | 1 +
>>   13 files changed, 13 insertions(+)
>>
>> diff --git a/gcc/testsuite/gcc.target/riscv/fmax.c b/gcc/testsuite/gcc.target/riscv/fmax.c
>> index c71d35c9f9dc..e1b7fa8f918c 100644
>> --- a/gcc/testsuite/gcc.target/riscv/fmax.c
>> +++ b/gcc/testsuite/gcc.target/riscv/fmax.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/fmaxf.c b/gcc/testsuite/gcc.target/riscv/fmaxf.c
>> index f9980166887a..8da0513dc8f6 100644
>> --- a/gcc/testsuite/gcc.target/riscv/fmaxf.c
>> +++ b/gcc/testsuite/gcc.target/riscv/fmaxf.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>>
>>   float
>> diff --git a/gcc/testsuite/gcc.target/riscv/fmin.c b/gcc/testsuite/gcc.target/riscv/fmin.c
>> index 9634abd19af8..01993d49bc21 100644
>> --- a/gcc/testsuite/gcc.target/riscv/fmin.c
>> +++ b/gcc/testsuite/gcc.target/riscv/fmin.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/fminf.c b/gcc/testsuite/gcc.target/riscv/fminf.c
>> index 9a3687be3092..32ce363e10d8 100644
>> --- a/gcc/testsuite/gcc.target/riscv/fminf.c
>> +++ b/gcc/testsuite/gcc.target/riscv/fminf.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-fno-finite-math-only -fsigned-zeros -fno-signaling-nans -dp" } */
>>
>>   float
>> diff --git a/gcc/testsuite/gcc.target/riscv/pr105666.c b/gcc/testsuite/gcc.target/riscv/pr105666.c
>> index 904f3bc0763f..dd996eec8efc 100644
>> --- a/gcc/testsuite/gcc.target/riscv/pr105666.c
>> +++ b/gcc/testsuite/gcc.target/riscv/pr105666.c
>> @@ -6,6 +6,7 @@
>>      spilling to stack.  */
>>
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-march=rv64g -ffast-math" } */
>>
>>   #define NITER 4
>> diff --git a/gcc/testsuite/gcc.target/riscv/smax-ieee.c b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
>> index 3a98aeb45add..2dbccefe2f4d 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smax-ieee.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smax-ieee.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/smax.c b/gcc/testsuite/gcc.target/riscv/smax.c
>> index d806c632faed..1092cc49f25e 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smax.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smax.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
>> index 6cf23d789b6e..31b9bfa4d91e 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smaxf-ieee.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>>
>>   float
>> diff --git a/gcc/testsuite/gcc.target/riscv/smaxf.c b/gcc/testsuite/gcc.target/riscv/smaxf.c
>> index d6a7a7f84cf7..aa1f22b45a17 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smaxf.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smaxf.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>>
>>   float
>> diff --git a/gcc/testsuite/gcc.target/riscv/smin-ieee.c b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
>> index c0a148c37cf1..ea36c2de89fe 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smin-ieee.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smin-ieee.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/smin.c b/gcc/testsuite/gcc.target/riscv/smin.c
>> index e325e9a1c7cc..d07a0fdc1525 100644
>> --- a/gcc/testsuite/gcc.target/riscv/smin.c
>> +++ b/gcc/testsuite/gcc.target/riscv/smin.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>>
>>   double
>> diff --git a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
>> index 353e7a18704a..427617af08f1 100644
>> --- a/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
>> +++ b/gcc/testsuite/gcc.target/riscv/sminf-ieee.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fsigned-zeros -dp" } */
>>
>>   float
>> diff --git a/gcc/testsuite/gcc.target/riscv/sminf.c b/gcc/testsuite/gcc.target/riscv/sminf.c
>> index f0ba7b43bdf9..7df223e8bcad 100644
>> --- a/gcc/testsuite/gcc.target/riscv/sminf.c
>> +++ b/gcc/testsuite/gcc.target/riscv/sminf.c
>> @@ -1,4 +1,5 @@
>>   /* { dg-do compile } */
>> +/* { dg-require-effective-target hard_float } */
>>   /* { dg-options "-ffinite-math-only -fno-signed-zeros -dp" } */
>>
>>   float
>> --
>> 2.32.0
>>
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v3] RISC-V/testsuite: constraint some of tests to hard_float
  2022-07-14  4:29       ` Vineet Gupta
@ 2022-07-21  9:23         ` Kito Cheng
  0 siblings, 0 replies; 7+ messages in thread
From: Kito Cheng @ 2022-07-21  9:23 UTC (permalink / raw)
  To: Vineet Gupta, jiawei
  Cc: Maciej W . Rozycki, Philipp Tomsich, GCC Patches,
	Andrew Waterman, gnu-toolchain

> On 5/29/22 20:50, Kito Cheng via Gcc-patches wrote:
> > Committed, thanks!
>
> Can this be backported to gcc-12 please.

I want to say yes but 9ddd44b58649d1d ("RISC-V: Provide `fmin'/`fmax'
RTL pattern") only existing in the trunk, and
gcc.target/riscv/pr105666.c part has fixed[1] during backport, anyway
that remind me there is another patch for the testcase [1], and back
port for that to gcc 12 branch

[1] https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=e919fae01b89fa6d7fc742d140bb15dc2600becb;hp=682d238f32a2aca993747f9c1ddf2b6f4c5fb536
[2] https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=b18e5d7e5f9df69759f0fbc2bed91d5e51313e79

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-07-21  9:23 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-27  1:44 [PATCH] RISC-V/testsuite: constraint some of tests to hard_float Vineet Gupta
2022-05-27  1:56 ` [PATCH v2] " Vineet Gupta
2022-05-27  2:36   ` [PATCH v3] " Vineet Gupta
2022-05-30  3:50     ` Kito Cheng
2022-07-14  4:29       ` Vineet Gupta
2022-07-21  9:23         ` Kito Cheng
2022-06-01 11:51     ` Maciej W. Rozycki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).