This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][testsuite][ARM] Properly figure -mfloat-abi option for crypto tests
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Christophe Lyon <christophe dot lyon at linaro dot org>
- Date: Fri, 10 Jan 2014 11:31:11 +0000
- Subject: Re: [PATCH][testsuite][ARM] Properly figure -mfloat-abi option for crypto tests
- Authentication-results: sourceware.org; auth=none
- References: <52CED61A dot 9060006 at arm dot com>
On 09/01/14 17:02, Kyrill Tkachov wrote:
> Hi all,
>
> When adding the testsuite options for the crypto tests we need to make sure that
> don't end up adding -mfloat-abi=softfp to a hard-float target like
> arm-none-linux-gnueabihf. This patch adds that code to figure out which
> -mfpu/-mfloat-abi combination to use in a similar approach to the NEON tests.
>
> This patch addresses the same failures that Christophe mentioned in
> http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00375.html
> but with this patch we can get those tests to PASS on arm-none-linux-gnueabihf
> instead of being just UNSUPPORTED.
>
> Tested arm-none-linux-gnueabihf and arm-none-eabi.
>
> Ok for trunk?
>
> Thanks,
> Kyrill
>
>
> 2014-01-09 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * lib/target-supports.exp
> (check_effective_target_arm_crypto_ok_nocache): New.
> (check_effective_target_arm_crypto_ok): Use above procedure.
> (add_options_for_arm_crypto): Use et_arm_crypto_flags.
>
>
OK.
R.