ping?
On Sat, 5 Jul 2025 at 00:01, Christophe Lyon <christophe.l...@linaro.org> wrote: > > Like we do in other effective-targets, add > "-mcpu=unset -march=armv8-a" > directly when setting et_arm_v8_neon_flags in arm_v8_neon_ok_nocache, > to avoid having to add these two flags in all users of arm_v8_neon_ok. > > This avoids duplication and possible typos / oversights. > > gcc/testsuite/ChangeLog: > * lib/target-supports.exp > (check_effective_target_arm_v8_neon_ok_nocache): Add "-mcpu=unset > -march=armv8-a" to et_arm_v8_neon_flags. > (add_options_for_vect_early_break): Remove useless "-mcpu=unset > -march=armv8-a". > (add_options_for_arm_v8_neon): Likewise. > --- > gcc/testsuite/lib/target-supports.exp | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/gcc/testsuite/lib/target-supports.exp > b/gcc/testsuite/lib/target-supports.exp > index 2581c8d5b01..5d06d70c017 100644 > --- a/gcc/testsuite/lib/target-supports.exp > +++ b/gcc/testsuite/lib/target-supports.exp > @@ -4597,7 +4597,7 @@ proc add_options_for_vect_early_break { flags } { > > if { [check_effective_target_arm_v8_neon_ok] } { > global et_arm_v8_neon_flags > - return "$flags $et_arm_v8_neon_flags -mcpu=unset -march=armv8-a" > + return "$flags $et_arm_v8_neon_flags" > } > > if { [check_effective_target_sse4] } { > @@ -5420,7 +5420,7 @@ proc add_options_for_arm_v8_neon { flags } { > return "$flags" > } > global et_arm_v8_neon_flags > - return "$flags $et_arm_v8_neon_flags -mcpu=unset -march=armv8-a" > + return "$flags $et_arm_v8_neon_flags" > } > > # Add the options needed for ARMv8.1 Adv.SIMD. Also adds the ARMv8 NEON > @@ -5879,7 +5879,7 @@ proc check_effective_target_arm_v8_neon_ok_nocache { } { > __asm__ volatile ("vrintn.f32 q0, q0"); > } > } "$flags -mcpu=unset -march=armv8-a"] } { > - set et_arm_v8_neon_flags $flags > + set et_arm_v8_neon_flags "$flags -mcpu=unset -march=armv8-a" > return 1 > } > } > -- > 2.34.1 >