[cfe-commits] [patch] Fix order of operands in NEON vfma intrinsic

Tim Northover Tim.Northover at arm.com
Mon Jan 14 05:37:59 PST 2013


Hi,

While applying some review comments for the AArch64 code, I noticed that the 
arguments are marshalled incorrectly when converting a NEON vfma intrinsic 
into an LLVM fma version.

The NEON version expects the modified (accumulator) argument to come first, 
however the LLVM version follows libm's semantics in putting the accumulator 
last.

This patch should fix the issue. Ok to commit?

Cheers.

Tim.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-fma-order.diff
Type: text/x-patch
Size: 1714 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130114/5de9e5df/attachment.bin>


More information about the cfe-commits mailing list