commit 28c7e41c0942877bf058470dc1f3f7ecfed43dba
author Sanjay Patel <spatel@rotateright.com> Wed Aug 01 23:05:55 2018 +0000
committer Sanjay Patel <spatel@rotateright.com> Wed Aug 01 23:05:55 2018 +0000
tree bdad7c974643074b9b673a82cdf260474ddab1d2
parent a30a6d2c29f59ffdf75a9e2c3dc53b2c93406f76

[InstSimplify] move minnum/maxnum with same arg fold from instcombine

llvm-svn: 338652

llvm/lib/Analysis/InstructionSimplify.cpp

diff --git a/llvm/lib/Analysis/InstructionSimplify.cpp b/llvm/lib/Analysis/InstructionSimplify.cpp
index 7fc7c15..b1601bf 100644
--- a/llvm/lib/Analysis/InstructionSimplify.cpp
+++ b/llvm/lib/Analysis/InstructionSimplify.cpp
@@ -4764,6 +4764,9 @@
     break;
   case Intrinsic::maxnum:
   case Intrinsic::minnum:
+    // If the arguments are the same, this is a no-op.
+    if (Op0 == Op1) return Op0;
+
     // If one argument is NaN, return the other argument.
     if (match(Op0, m_NaN())) return Op1;
     if (match(Op1, m_NaN())) return Op0;

llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp

diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
index cbfbd8a..77c5a06 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
@@ -1137,10 +1137,6 @@
   Value *Arg0 = II.getArgOperand(0);
   Value *Arg1 = II.getArgOperand(1);
 
-  // fmin(x, x) -> x
-  if (Arg0 == Arg1)
-    return Arg0;
-
   const auto *C1 = dyn_cast<ConstantFP>(Arg1);
 
   // fmin(x, nan) -> x