This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM] Error out of arm_neon.h if compiling for soft-float ABI
- From: Christophe Lyon <christophe dot lyon at linaro dot org>
- To: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Wed, 2 Mar 2016 16:10:08 +0100
- Subject: Re: [PATCH][ARM] Error out of arm_neon.h if compiling for soft-float ABI
- Authentication-results: sourceware.org; auth=none
- References: <56D4847C dot 1000305 at foss dot arm dot com>
On 29 February 2016 at 18:48, Kyrill Tkachov
<kyrylo.tkachov@foss.arm.com> wrote:
> Hi all,
>
> Now that we've moved to pragmas guarding the various intrinsics in
> arm_neon.h I think we should still
> throw a #error if someone tries to include the header while compiling for
> -mfloat-abi=soft.
>
> This gives a more helpful error message when someone has a compiler
> configured --with-float=soft
> and forgets to add an appropriate -mfloat-abi option on the command line.
> Currently we'll just give tons of error messages whereas with this patch we
> just
> show a simple clean message.
>
> Tested on arm. This could be argued to be a user experience regression fix.
Agreed :)
> Ok for trunk?
>
> Thanks,
> Kyrill
>
> 2016-02-29 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * config/arm/arm_neon.h: Show error if using with soft-float ABI.