[PATCH] D50714: [InstCombine] Fold Select with binary op - FP opcodes

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineSelect.cpp:71
+  else if (FCmpInst::isEquality(Pred))
+    IsEq = Pred == FCmpInst::FCMP_OEQ || Pred == FCmpInst::FCMP_UEQ;
+  else
----------------
spatel wrote:
> Did you step through the scenarios where X is NAN? I don't think the transform is valid for both predicates (similarly for ONE/UNE).
Ok, so probably not worth to with FP opcodes? if it is so complicated.. Alive has some problems with float?


https://reviews.llvm.org/D50714