[PATCH] D33186: [InstCombine] Canonicalize clamp of float types to minmax in fast mode.

[James Molloy via Phabricator via llvm-commits]

jmolloy added a comment.

Hi,

> On the other hand, some backends (e.g. X86) have only

FMIN/FMAX nodes that do not care about NaNS and the former NAN/NUM
nodes are illegal thus selection is not happening.

For my own reference so I can more effectively review this, could you please explain this a bit more? What is the defined behaviour for such instructions when given a NaN?


https://reviews.llvm.org/D33186