[PATCH] Minor fix for FMA3 instruction encoding in JIT

Nadav Rotem nrotem at apple.com
Tue Aug 20 17:17:45 PDT 2013


LGTM. 

On Aug 20, 2013, at 5:06 PM, Chris Bieneman <cbieneman at apple.com> wrote:

> Hi,
> 
> This patch fixes an issue with FMA3 instruction encoding.
> 
> In LLVM FMA3 operands are dst, src1, src2, src3, however dst is not encoded as it is always src1. This was causing the encoding of the operands to be off by one.
> 
> I’ve also put together a second small patch which adds a test case and extends lit to have an ‘fma3’ available_feature and provides a test case for using the fma intrinsics in lli.
> 
> Thanks,
> -Chris
> 
> <fma3-jit-test.diff><fma3-jit-patch.diff>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list