Hi kyrylo, > ________________________________________ > From: Kyrylo Tkachov <kyrylo.tkac...@arm.com> > Sent: Tuesday, April 7, 2020 3:07 PM > To: Dennis Zhang; gcc-patches@gcc.gnu.org > Cc: nd; Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH][Arm][2/4] Custom Datapath Extension intrinsics: > instructions using FPU/MVE S/D registers > > Hi Dennis, > > > -----Original Message----- > > From: Dennis Zhang <dennis.zh...@arm.com> > > Sent: 07 April 2020 13:31 > > To: gcc-patches@gcc.gnu.org > > Cc: nd <n...@arm.com>; Richard Earnshaw <richard.earns...@arm.com>; > > Ramana Radhakrishnan <ramana.radhakrish...@arm.com>; Kyrylo > > Tkachov <kyrylo.tkac...@arm.com> > > Subject: Re: [PATCH][Arm][2/4] Custom Datapath Extension intrinsics: > > instructions using FPU/MVE S/D registers > > > > Hi all, > > > > This patch is updated to support DImode for vfp target as required by CDE. > > Changelog is updated as following. > > > > Is this ready for commit please? > > This is ok. > Has the first patch been updated and committed yet? > Thanks, > Kyrill >
This patch has been committed as 07b9bfd02b88cad2f6b3f50ad610dd75cb989ed3. Many thanks Dennis > > > > Cheers > > Dennis > > > > gcc/ChangeLog: > > > > 2020-04-07 Dennis Zhang <dennis.zh...@arm.com> > > Matthew Malcomson <matthew.malcom...@arm.com> > > > > * config/arm/arm-builtins.c (CX_IMM_QUALIFIERS): New macro. > > (CX_UNARY_QUALIFIERS, CX_BINARY_QUALIFIERS): Likewise. > > (CX_TERNARY_QUALIFIERS): Likewise. > > (ARM_BUILTIN_CDE_PATTERN_START): Likewise. > > (ARM_BUILTIN_CDE_PATTERN_END): Likewise. > > (arm_init_acle_builtins): Initialize CDE builtins. > > (arm_expand_acle_builtin): Check CDE constant operands. > > * config/arm/arm.h (ARM_CDE_CONST_COPROC): New macro to set the > > range > > of CDE constant operand. > > * config/arm/arm.c (arm_hard_regno_mode_ok): Support DImode for > > TARGET_VFP_BASE. > > (ARM_VCDE_CONST_1, ARM_VCDE_CONST_2, ARM_VCDE_CONST_3): > > Likewise. > > * config/arm/arm_cde.h (__arm_vcx1_u32): New macro of ACLE interface. > > (__arm_vcx1a_u32, __arm_vcx2_u32, __arm_vcx2a_u32): Likewise. > > (__arm_vcx3_u32, __arm_vcx3a_u32, __arm_vcx1d_u64): Likewise. > > (__arm_vcx1da_u64, __arm_vcx2d_u64, __arm_vcx2da_u64): Likewise. > > (__arm_vcx3d_u64, __arm_vcx3da_u64): Likewise. > > * config/arm/arm_cde_builtins.def: New file. > > * config/arm/iterators.md (V_reg): New attribute of SI. > > * config/arm/predicates.md (const_int_coproc_operand): New. > > (const_int_vcde1_operand, const_int_vcde2_operand): New. > > (const_int_vcde3_operand): New. > > * config/arm/unspecs.md (UNSPEC_VCDE, UNSPEC_VCDEA): New. > > * config/arm/vfp.md (arm_vcx1<mode>): New entry. > > (arm_vcx1a<mode>, arm_vcx2<mode>, arm_vcx2a<mode>): Likewise. > > (arm_vcx3<mode>, arm_vcx3a<mode>): Likewise. > > > > gcc/testsuite/ChangeLog: > > > > 2020-04-07 Dennis Zhang <dennis.zh...@arm.com> > > > > * gcc.target/arm/acle/cde_v_1.c: New test. > > * gcc.target/arm/acle/cde_v_1_err.c: New test. > > * gcc.target/arm/acle/cde_v_1_mve.c: New test. > > > > > Hi all, > > > > > > This patch is updated as attached. > > > It's rebased to the top. Is it ready for commit please? > > > > > > Cheers > > > Dennis > > > > > > > Hi all, > > > > > > > > This patch is part of a series that adds support for the ARMv8.m Custom > > Datapath Extension (CDE). > > > > It enables the ACLE intrinsics calling VCX1<A>, VCX2<A>, and VCX3<A> > > instructions who work with FPU/MVE 32-bit/64-bit registers. > > > > > > > > This patch depends on the CDE feature patch: > > https://gcc.gnu.org/pipermail/gcc-patches/2020-March/541921.html > > > > It also depends on the MVE framework patch: > > https://gcc.gnu.org/pipermail/gcc-patches/2020-February/540415.html > > > > ISA has been announced at > > https://developer.arm.com/architectures/instruction-sets/custom- > > instructions > > > > > > > > Regtested and bootstrapped for arm-none-linux-gnueabi-armv8-m.main. > > > > > > > > Is it OK for commit please? > > > > > > > > Cheers > > > > Dennis