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

Tim Northover t.p.northover at gmail.com
Wed Jan 16 08:38:19 PST 2013


Ping?

Tim.

On Mon, Jan 14, 2013 at 1:37 PM, Tim Northover <Tim.Northover at arm.com> wrote:
> 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.
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-fma-order.diff
Type: application/octet-stream
Size: 1715 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130116/7a242be4/attachment.obj>


More information about the cfe-commits mailing list