[PATCH] D21137: Instcombile min/max intrinsics calls

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 11:08:55 PDT 2016


majnemer added inline comments.

================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:977-978
@@ +976,4 @@
+        IntrinsicInst *I = cast<IntrinsicInst>(Arg1);
+        if (I->hasNoNaNs())
+          return replaceInstUsesWith(*II, Arg0);
+        Instruction *Result = new FCmpInst(II, FCmpInst::FCMP_UNO, Arg0, Arg0);
----------------
Couldn't this case live in InstructionSimplify?


http://reviews.llvm.org/D21137





More information about the llvm-commits mailing list