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/vect-early-break-cbranch.c: Use
> effective-target arm_arch_v8a_hard.
>
> Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
> ---
> gcc/testsuite/gcc.target/arm/vect-early-break-cbranch.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.target/arm/vect-early-break-cbranch.c
> b/gcc/testsuite/gcc.target/arm/vect-early-break-cbranch.c
> index 334e064a239..fb12bfb3197 100644
> --- a/gcc/testsuite/gcc.target/arm/vect-early-break-cbranch.c
> +++ b/gcc/testsuite/gcc.target/arm/vect-early-break-cbranch.c
> @@ -2,7 +2,9 @@
> /* { dg-require-effective-target vect_early_break } */
I think this is technically redundant (it just checks for neon on armv8a), but
it's probably a good idea to keep it just in case it grows an additional check
at some point.
> /* { dg-require-effective-target arm_neon_ok } */
> /* { dg-require-effective-target arm32 } */
These two are, I think, redundant, so can be removed; the flags added below
will ensure they are true.
> -/* { dg-options "-O3 -march=armv8-a+simd -mfpu=auto -mfloat-abi=hard
> -fno-schedule-insns -fno-reorder-blocks -fno-schedule-insns2" } */
> +/* { dg-require-effective-target arm_arch_v8a_hard_ok } */
> +/* { dg-options "-O3 -fno-schedule-insns -fno-reorder-blocks
> -fno-schedule-insns2" } */
> +/* { dg-add-options arm_arch_v8a_hard } */
> /* { dg-final { check-function-bodies "**" "" "" } } */
>
> #define N 640
OK with that change.
R.
