Re: [PATCH, rs6000] Add builtin support for power8 32-bit Altivec multiply insns

2013-08-02 Thread Michael Meissner
On Fri, Aug 02, 2013 at 04:31:37PM -0500, Peter Bergner wrote: > On Fri, 2013-08-02 at 17:13 -0400, David Edelsohn wrote: > > The builtins and patterns seem to be in a random order. Sometimes > > signed is first and sometimes unsigned is first. Please make the > > ordering of the names consistent.

Re: [PATCH, rs6000] Add builtin support for power8 32-bit Altivec multiply insns

2013-08-02 Thread Peter Bergner
On Fri, 2013-08-02 at 17:13 -0400, David Edelsohn wrote: > The builtins and patterns seem to be in a random order. Sometimes > signed is first and sometimes unsigned is first. Please make the > ordering of the names consistent. Ok, I can clean that up. > If these operations are multiplies, why

Re: [PATCH, rs6000] Add builtin support for power8 32-bit Altivec multiply insns

2013-08-02 Thread David Edelsohn
On Thu, Aug 1, 2013 at 2:21 PM, Peter Bergner wrote: > This patch adds builtin support for the new 32-bit Altivec multiply > instructions that were added in ISA 2.07 (ie, POWER8). > > This passed bootstrap and regtesting with no errors. Ok for mainline? Peter, The builtins and patterns seem to