From: Tamar Christina Date: Fri, 15 Feb 2019 17:17:17 +0000 (+0000) Subject: Arm: Remove alternative from neon_softfp_fp16 directive. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=75e2abe6f9a3b423e952b5741fcb2787834e19f4;p=gcc.git Arm: Remove alternative from neon_softfp_fp16 directive. There's a bit of a disconnect between the feature flags that don't test the fpu and ones that do when the test itself also forces an architecture. The forcing of the architecture would change the defaults and without explicitly giving the correct fpu again the test would fail. I don't see a good way to solve this problem, really the feature tests should ideally contain the extra options the test adds too, but for this specific case it can be solved by always testing the fpu explicitly. Committed under the GCC obvious gcc/testsuite/ChangeLog: * lib/target-supports.exp (check_effective_target_arm_neon_softfp_fp16_ok_nocache): Drop non-fpu checking alternative. From-SVN: r268943 --- diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 86383671f06..e279a8e54ea 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2019-02-15 Tamar Christina + + * lib/target-supports.exp + (check_effective_target_arm_neon_softfp_fp16_ok_nocache): Drop non-fpu + checking alternative. + 2019-02-15 Eric Botcazou * gcc.target/sparc/struct-ret-check-1.c: Add -fno-pie option. diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp index a56796393a5..94cb319bc12 100644 --- a/gcc/testsuite/lib/target-supports.exp +++ b/gcc/testsuite/lib/target-supports.exp @@ -3797,7 +3797,6 @@ proc check_effective_target_arm_neon_softfp_fp16_ok_nocache { } { if { [check_effective_target_arm32] && [check_effective_target_arm_neon_ok] } { foreach flags {"-mfpu=neon-fp16 -mfloat-abi=softfp" - "-mfloat-abi=softfp -mfp16-format=ieee" "-mfpu=neon-fp16 -mfloat-abi=softfp -mfp16-format=ieee"} { if { [check_no_compiler_messages_nocache arm_neon_softfp_fp16_ok object { #include "arm_neon.h"