On 19 October 2016 at 11:36, Tamar Christina <tamar.christ...@arm.com> wrote: > Hi All, > > This patch implements the vmaxnmQ_ST and vminnmQ_ST intrinsics. The > current builtin registration code is deficient since it can't access > standard pattern names, to which vmaxnmQ_ST and vminnmQ_ST map > directly. Thus, to enable the vectoriser to have access to these > intrinsics, we implement them using builtin functions, which we > expand to the proper standard pattern using a define_expand. > > This patch also implements the __ARM_FEATURE_NUMERIC_MAXMIN macro, > which is defined when __ARM_ARCH >= 8, and which enables the > intrinsics. > > Regression tested on arm-none-eabi and no regressions. > > This patch is a rework of a previous patch: > https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01971.html > > OK for trunk? > > Thanks, > Tamar > > --- > > gcc/ > > 2016-10-19 Bilyan Borisov <bilyan.bori...@arm.com> > Tamar Christina <tamar.christ...@arm.com> > > * config/arm/arm-c.c (arm_cpu_builtins): New macro definition. > * config/arm/arm_neon.h (vmaxnm_f32): New intrinsinc. > (vmaxnmq_f32): Likewise. > (vminnm_f32): Likewise. > (vminnmq_f32): Likewise. > * config/arm/arm_neon_builtins.def (vmaxnm): New builtin. > (vminnm): Likewise. > * config/arm/neon.md (neon_<fmaxmin_op><mode>, VCVTF): New > expander. > > gcc/testsuite/ > > 2016-10-19 Bilyan Borisov <bilyan.bori...@arm.com> > > * gcc.target/arm/simd/vmaxnm_f32_1.c: New. > * gcc.target/arm/simd/vmaxnmq_f32_1.c: Likewise. > * gcc.target/arm/simd/vminnm_f32_1.c: Likewise. > * gcc.target/arm/simd/vminnmq_f32_1.c: Likewise. >
I think you forgot to attach the new tests. Christophe
