On 19/11/2024 10:23, Torbjörn SVENSSON wrote:
> Update test cases to use -mcpu=unset/-march=unset feature introduced in
> r15-3606-g7d6c6a0d15c.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/arm/acle/crc_hf_1.c: Use effective-target
> arm_arch_v8a_hard and added option "-mcpu=unset".
>
> Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
> ---
> gcc/testsuite/gcc.target/arm/acle/crc_hf_1.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.target/arm/acle/crc_hf_1.c
> b/gcc/testsuite/gcc.target/arm/acle/crc_hf_1.c
> index f1de2bdffee..378f5f04217 100644
> --- a/gcc/testsuite/gcc.target/arm/acle/crc_hf_1.c
> +++ b/gcc/testsuite/gcc.target/arm/acle/crc_hf_1.c
> @@ -2,8 +2,9 @@
> break CRC intrinsics. */
>
> /* { dg-do compile } */
> +/* { dg-require-effective-target arm_arch_v8a_hard_ok } */
> /* { dg-require-effective-target arm_hard_vfp_ok } */
> -/* { dg-additional-options "-mfloat-abi=hard -march=armv8-a+simd+crc" } */
> +/* { dg-additional-options "-mfloat-abi=hard -mcpu=unset
> -march=armv8-a+simd+crc" } */
>
> #include <arm_acle.h>
>
I'd be more inclined to add
v8a_crc_hard "-march=armv8-a+crc+simd -mfpu=auto -mfloat-abi=hard"
"__ARM_ARCH_8A__ && __ARM_FEATURE_CRC32"
to the table in target-supports.exp, then this test becomes
dg-do compile
dg-require-effective-target arm_arch_v8a_crc_hard_ok
dg-add-options arm_arch_v8a_crc_hard
(we don't even need the arm_hard_vfp_ok then).
OK with that change.
