[PATCH] [ARM] generate VMAXNM/VMINNM for a compare followed by a select (in safe math mode too)

[Artyom Skrobov]

Thank you Renato!

Committed revision 236590.


-----Original Message-----
From: Renato Golin [mailto:renato.golin at linaro.org] 
Sent: 06 May 2015 10:42
To: Artyom Skrobov
Cc: llvm-commits
Subject: Re: [PATCH] [ARM] generate VMAXNM/VMINNM for a compare followed by a select (in safe math mode too)

On 5 May 2015 at 20:48, Artyom Skrobov <artyom.skrobov at arm.com> wrote:
> Yes, I did put together some back-of-the-envelope tables of the possibilities, to check that the semantics are preserved by each of the switch blocks.

Excellent, thanks!


> What matters for the transformability is whether the operands are NaN, it doesn't matter whether they're constants.
> You're right that my comments there are slightly inaccurate -- they should say "the non-NaN should be RHS" and "the non-NaN should be LHS" instead.
> Thanks for catching that!  I'll fix the comments.

Makes sense.

cheers,
--renato