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

[Anton Korobeynikov]

Tim,

Looks obvious enough to be comitted as-is :)

On Wed, Jan 16, 2013 at 8:38 PM, Tim Northover <t.p.northover at gmail.com> wrote:
> 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
>>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University