RE: [PATCH 5/5][AArch64] fp16fml support

2018-01-10 Thread Michael Collison
atches <gcc-patches@gcc.gnu.org>; nd <n...@arm.com> Subject: Re: [PATCH 5/5][AArch64] fp16fml support On Tue, Jan 09, 2018 at 06:28:09PM +, Michael Collison wrote: > Patch updated per Richard's comments. Ok for trunk? This patch adds a lot of code, much of which looks like it ough

Re: [PATCH 5/5][AArch64] fp16fml support

2018-01-10 Thread James Greenhalgh
January 4, 2018 8:02 AM > To: Michael Collison <michael.colli...@arm.com> > Cc: GCC Patches <gcc-patches@gcc.gnu.org>; nd <n...@arm.com> > Subject: Re: [PATCH 5/5][AArch64] fp16fml support > > Hi Michael, > > Not a review of the full patch, just a comment ab

RE: [PATCH 5/5][AArch64] fp16fml support

2018-01-09 Thread Michael Collison
lt;n...@arm.com> Subject: Re: [PATCH 5/5][AArch64] fp16fml support Hi Michael, Not a review of the full patch, just a comment about the patterns: Michael Collison <michael.colli...@arm.com> writes: > +(define_expand "aarch64_fmll_lane_lowv2sf" > + [(set (match_operand:V2SF 0 &qu

Re: [PATCH 5/5][AArch64] fp16fml support

2018-01-04 Thread Richard Sandiford
Hi Michael, Not a review of the full patch, just a comment about the patterns: Michael Collison writes: > +(define_expand "aarch64_fmll_lane_lowv2sf" > + [(set (match_operand:V2SF 0 "register_operand" "") > + (unspec:V2SF [(match_operand:V2SF 1 "register_operand"