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 <a...@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: >