On Thu, Aug 1, 2013 at 2:21 PM, Peter Bergner <berg...@vnet.ibm.com> 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 be in a random order. Sometimes signed is first and sometimes unsigned is first. Please make the ordering of the names consistent. If these operations are multiplies, why are they described as UNSPEC instead of MULT with SELECT? Thanks, David
