On 2024-11-05 13:54, Richard Earnshaw (lists) wrote:
On 05/11/2024 07:49, Torbjörn SVENSSON wrote:
Ok for trunk and releases/gcc-14?
--
The test case assumes that -mfp16-format=alternative is accepted for the
target, but not all targets support this flag. One such target is
Cortex-M85 that does support FP16, but not the alternative format.
gcc/testsuite/ChangeLog:
* gcc.target/arm/pr98636.c: Use effective-target
arm_fp16_alternative.
Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
---
gcc/testsuite/gcc.target/arm/pr98636.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/testsuite/gcc.target/arm/pr98636.c
b/gcc/testsuite/gcc.target/arm/pr98636.c
index 559f9a26c1e..76bf7ac6589 100644
--- a/gcc/testsuite/gcc.target/arm/pr98636.c
+++ b/gcc/testsuite/gcc.target/arm/pr98636.c
@@ -1,5 +1,6 @@
/* { dg-do compile } */
/* { dg-require-effective-target arm_softfp_ok } */
+/* { dg-require-effective-target arm_fp16_alternative_ok } */
/* { dg-options "-mfp16-format=alternative -mfloat-abi=softfp" } */
I think you also need dg-add-options arm_fp16_alternative, since the OK test
checks various option combinations to see if they work. So
/* { dg-options "-mfloat-abi=softfp" } */
/* { dg-add-options arm_fp16_alternative } */
OK with that change (assuming it works).
It works, so pushed as r15-4965-g4602f628f72 and r14.2.0-362-g232d539f25b.
Kind regards,
Torbjörn
