[llvm-commits] [llvm] [PATCH] PowerPC: Fix Altivec multiply-add zero sign
Adhemerval Zanella
azanella at linux.vnet.ibm.com
Fri Nov 30 05:07:23 PST 2012
On 11/29/2012 05:37 PM, Hal Finkel wrote:
>
> Generally looks good to me. One quick comment:
>
> +; CHECK: VSPLTI:
> +; CHECK: vsplti
>
> I recommend writing this as:
> +; CHECK: @VSPLTI
> +; CHECK: vsplti
>
> to make it clear that the first check is to match the function definition.
>
> -Hal
>
I committed as revision 168998 with your comment.
More information about the llvm-commits
mailing list