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).
R.
>
> #pragma GCC push_options
