[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