[llvm] cd44cc8 - [InstCombine] remove unused function argument; NFC

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 12 08:47:30 PDT 2021


Author: Sanjay Patel
Date: 2021-08-12T11:47:25-04:00
New Revision: cd44cc86e34ebd29410c3452e44e241e281a7288

URL: https://github.com/llvm/llvm-project/commit/cd44cc86e34ebd29410c3452e44e241e281a7288
DIFF: https://github.com/llvm/llvm-project/commit/cd44cc86e34ebd29410c3452e44e241e281a7288.diff

LOG: [InstCombine] remove unused function argument; NFC

This was just added with 6de1dbbd09c1 , and I missed
pulling the extra arg from the final revision.

Added: 
    

Modified: 
    llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
index bda8c2531731..d6ccf0fcc708 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
@@ -796,13 +796,12 @@ static Instruction *foldClampRangeOfTwo(IntrinsicInst *II,
 }
 
 /// Reduce a sequence of min/max intrinsics with a common operand.
-static Instruction *factorizeMinMaxTree(IntrinsicInst *II,
-                                        InstCombiner::BuilderTy &Builder) {
+static Instruction *factorizeMinMaxTree(IntrinsicInst *II) {
   // Match 3 of the same min/max ops. Example: umin(umin(), umin()).
   auto *LHS = dyn_cast<IntrinsicInst>(II->getArgOperand(0));
   auto *RHS = dyn_cast<IntrinsicInst>(II->getArgOperand(1));
   Intrinsic::ID MinMaxID = II->getIntrinsicID();
-  if (!LHS || !RHS || LHS->getIntrinsicID() !=  MinMaxID ||
+  if (!LHS || !RHS || LHS->getIntrinsicID() != MinMaxID ||
       RHS->getIntrinsicID() != MinMaxID ||
       (!LHS->hasOneUse() && !RHS->hasOneUse()))
     return nullptr;
@@ -1111,7 +1110,7 @@ Instruction *InstCombinerImpl::visitCallInst(CallInst &CI) {
         if (Instruction *R = FoldOpIntoSelect(*II, Sel))
           return R;
 
-    if (Instruction *NewMinMax = factorizeMinMaxTree(II, Builder))
+    if (Instruction *NewMinMax = factorizeMinMaxTree(II))
       return NewMinMax;
 
     break;


        


More information about the llvm-commits mailing list