On 20/03/2025 16:15, Christophe Lyon wrote:
> Like a previous patch, replace "" with -mfpu=auto to match the
> intended effect of -march=armv8.2-a+fp16.
>
> No visible change because the effect is masked by other effective
> targets used in the tests, done for consistency.
>
> gcc/testsuite/
> * lib/target-supports.exp
> (check_effective_target_arm_v8_2a_bf16_neon_ok_nocache):
> Conditionally use -mfpu=auto.
Again, see comments on patch 6.
R.
> ---
> gcc/testsuite/lib/target-supports.exp | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/testsuite/lib/target-supports.exp
> b/gcc/testsuite/lib/target-supports.exp
> index c2df22d2255..587db04b95e 100644
> --- a/gcc/testsuite/lib/target-supports.exp
> +++ b/gcc/testsuite/lib/target-supports.exp
> @@ -6661,6 +6661,7 @@ proc
> check_effective_target_arm_v8_2a_fp16_neon_ok_nocache { } {
> global et_arm_v8_2a_fp16_neon_flags
> set et_arm_v8_2a_fp16_neon_flags ""
> set cpu_unset ""
> + set fpu_auto ""
>
> if { ![istarget arm*-*-*] && ![istarget aarch64*-*-*] } {
> return 0;
> @@ -6668,12 +6669,15 @@ proc
> check_effective_target_arm_v8_2a_fp16_neon_ok_nocache { } {
>
> if { [istarget arm*-*-*] } {
> set cpu_unset "-mcpu=unset"
> + set fpu_auto "-mfpu=auto"
> }
>
> # Iterate through sets of options to find the compiler flags that
> # need to be added to the -march option.
> - foreach flags {"" "-mfpu=neon-fp-armv8" "-mfloat-abi=softfp" \
> - "-mfpu=neon-fp-armv8 -mfloat-abi=softfp"} {
> + foreach flags [list "$fpu_auto" \
> + "-mfpu=neon-fp-armv8" \
> + "-mfloat-abi=softfp" \
> + "-mfpu=neon-fp-armv8 -mfloat-abi=softfp"] {
> if { [check_no_compiler_messages_nocache \
> arm_v8_2a_fp16_neon_ok object {
> #if !defined (__ARM_FEATURE_FP16_VECTOR_ARITHMETIC)
