Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

[Richard Earnshaw <rearnsha@arm.com>]

On 15/09/14 11:56, Andrew Stubbs wrote:
> On 15/09/14 10:46, Richard Earnshaw wrote:
>> Hmm, I wonder if arm_override_options should reject neon + (arch < 7).
> 
> Is this more to your taste?
> 

Yep, that's fine.

> Andrew
> 
> P.S. arm_override_options was renamed in 2010.

I'm getting old :-(

R.

> 
> 
> arm_neon_ok-2.patch
> 
> 
> 2014-09-15  Andrew Stubbs  <ams@codesourcery.com>
> 
> 	* gcc/config/arm/arm.c (arm_option_override): Reject -mfpu=neon
> 	when architecture is older than ARMv7.
> 
> Index: gcc/config/arm/arm.c
> ===================================================================
> --- gcc/config/arm/arm.c	(revision 215228)
> +++ gcc/config/arm/arm.c	(working copy)
> @@ -2845,6 +2845,9 @@
>  
>    arm_fpu_desc = &all_fpus[arm_fpu_index];
>  
> +  if (TARGET_NEON && !arm_arch7)
> +    error ("target CPU does not support NEON");
> +
>    switch (arm_fpu_desc->model)
>      {
>      case ARM_FP_MODEL_VFP:
>