[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