From: Dan Gohman Date: Sat, 29 Aug 2009 23:39:38 +0000 (+0000) Subject: Remove an unnecessary Context argument. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=14ef4f0a4c1e5b157b21709b4ee9e65d4ec760cb;p=oota-llvm.git Remove an unnecessary Context argument. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@80454 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 62ac9c91e20..f21decea0fb 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -412,7 +412,7 @@ X("instcombine", "Combine redundant instructions"); // getComplexity: Assign a complexity or rank value to LLVM Values... // 0 -> undef, 1 -> Const, 2 -> Other, 3 -> Arg, 3 -> Unary, 4 -> OtherInst -static unsigned getComplexity(LLVMContext *Context, Value *V) { +static unsigned getComplexity(Value *V) { if (isa(V)) { if (BinaryOperator::isNeg(V) || BinaryOperator::isFNeg(V) || @@ -512,8 +512,7 @@ static bool ValueRequiresCast(Instruction::CastOps opcode, const Value *V, // bool InstCombiner::SimplifyCommutative(BinaryOperator &I) { bool Changed = false; - if (getComplexity(Context, I.getOperand(0)) < - getComplexity(Context, I.getOperand(1))) + if (getComplexity(I.getOperand(0)) < getComplexity(I.getOperand(1))) Changed = !I.swapOperands(); if (!I.isAssociative()) return Changed; @@ -551,8 +550,7 @@ bool InstCombiner::SimplifyCommutative(BinaryOperator &I) { /// so that theyare listed from right (least complex) to left (most complex). /// This puts constants before unary operators before binary operators. bool InstCombiner::SimplifyCompare(CmpInst &I) { - if (getComplexity(Context, I.getOperand(0)) >= - getComplexity(Context, I.getOperand(1))) + if (getComplexity(I.getOperand(0)) >= getComplexity(I.getOperand(1))) return false; I.swapOperands(); // Compare instructions are not associative so there's nothing else we can do.