> -----Original Message----- > From: Richard Earnshaw > Sent: 17 December 2012 14:13 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan > Subject: Re: [PATCH][ARM][testsuite] Add testsuite options for AArch32 > NEON > > On 11/12/12 12:30, Kyrylo Tkachov wrote: > > Hi all, > > > > Since the new AArch32 NEON instructions in arm_neon.h are predicated > on > > __ARM_ARCH 8 the testsuite add_options procedure should also include > > -march=armv8-a to make these instructions available. > > > > This makes the new vrnd* tests in gcc.target/arm/neon PASS in > situations > > where armv8-a is not the default target. > > No regressions otherwise on arm-none-eabi. > > > > Ok for trunk? > > > > OK.
Thanks for the review, Richard. Committed as r194556. > > Don't you need to do the same thing for add_options_for_arm_v8_vfp? Shouldn't need that for the scalar v8 FP tests. The -march=armv8-a for the NEON intrinsic tests is only needed because the intrinsics in arm_neon.h are only available when __ARM_ARCH is >= 8. The scalar floating point instructions only depend on the fpu and not the architecture. > > R. Thanks, Kyrill
