Here is the updated patch. The changes are minor, so let me know if 
there is anything else to fix or if it can be committed.

Thank you,
Delia

On 1/30/20 2:55 PM, Kyrill Tkachov wrote:
> Hi Delia,
> 
> 
> On 1/28/20 4:44 PM, Delia Burduv wrote:
>> Ping.
>> ------------------------------------------------------------------------
>> *From:* Delia Burduv <delia.burduv@arm.com>
>> *Sent:* 22 January 2020 17:26
>> *To:* gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>
>> *Cc:* nickc@redhat.com <nickc@redhat.com>; Richard Earnshaw 
>> <Richard.Earnshaw@arm.com>; Ramana Radhakrishnan 
>> <Ramana.Radhakrishnan@arm.com>; Kyrylo Tkachov <Kyrylo.Tkachov@arm.com>
>> *Subject:* Re: [GCC][PATCH][AArch32] ACLE intrinsics bfloat16 vmmla 
>> and vfma<b/t> for AArch32 AdvSIMD
>> Ping.
>>
>> I have read Richard Sandiford's comments on the AArch64 patches and I
>> will apply what is relevant to this patch as well. Particularly, I will
>> change the tests to use the exact input and output registers and I will
>> change the types of the rtl patterns.
> 
> 
> Please send the updated patches so that someone can commit them for you 
> once they're reviewed.
> 
> Thanks,
> 
> Kyrill
> 
> 
>>
>> On 12/20/19 6:44 PM, Delia Burduv wrote:
>> > This patch adds the ARMv8.6 ACLE intrinsics for vmmla, vfmab and vfmat
>> > as part of the BFloat16 extension.
>> > (https://developer.arm.com/docs/101028/latest.)
>> > The intrinsics are declared in arm_neon.h and the RTL patterns are
>> > defined in neon.md.
>> > Two new tests are added to check assembler output and lane indices.
>> >
>> > This patch depends on the Arm back-end patche.
>> > (https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01448.html)
>> >
>> > Tested for regression on arm-none-eabi and armeb-none-eabi. I don't 
>> have
>> > commit rights, so if this is ok can someone please commit it for me?
>> >
>> > gcc/ChangeLog:
>> >
>> > 2019-11-12  Delia Burduv <delia.burduv@arm.com>
>> >
>> >      * config/arm/arm_neon.h (vbfmmlaq_f32): New.
>> >        (vbfmlalbq_f32): New.
>> >        (vbfmlaltq_f32): New.
>> >        (vbfmlalbq_lane_f32): New.
>> >        (vbfmlaltq_lane_f32): New.
>> >          (vbfmlalbq_laneq_f32): New.
>> >        (vbfmlaltq_laneq_f32): New.
>> >      * config/arm/arm_neon_builtins.def (vbfmmla): New.
>> >            (vbfmab): New.
>> >            (vbfmat): New.
>> >            (vbfmab_lane): New.
>> >            (vbfmat_lane): New.
>> >            (vbfmab_laneq): New.
>> >            (vbfmat_laneq): New.
>> >       * config/arm/iterators.md (BF_MA): New int iterator.
>> >            (bt): New int attribute.
>> >            (VQXBF): Copy of VQX with V8BF.
>> >            (V_HALF): Added V8BF.
>> >        * config/arm/neon.md (neon_vbfmmlav8hi): New insn.
>> >            (neon_vbfma<bt>v8hi): New insn.
>> >            (neon_vbfma<bt>_lanev8hi): New insn.
>> >            (neon_vbfma<bt>_laneqv8hi): New expand.
>> >            (neon_vget_high<mode>): Changed iterator to VQXBF.
>> >      * config/arm/unspecs.md (UNSPEC_BFMMLA): New UNSPEC.
>> >            (UNSPEC_BFMAB): New UNSPEC.
>> >            (UNSPEC_BFMAT): New UNSPEC.
>> >
>> > 2019-11-12  Delia Burduv <delia.burduv@arm.com>
>> >
>> >          * gcc.target/arm/simd/bf16_ma_1.c: New test.
>> >          * gcc.target/arm/simd/bf16_ma_2.c: New test.
>> >          * gcc.target/arm/simd/bf16_mmla_1.c: New test.