Tim,

Looks obvious enough to be comitted as-is :)

On Wed, Jan 16, 2013 at 8:38 PM, Tim Northover <[email protected]> wrote:
> Ping?
>
> Tim.
>
> On Mon, Jan 14, 2013 at 1:37 PM, Tim Northover <[email protected]> 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
>> [email protected]
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to