From e53e8c8e5888debe378c0cf35fcd4efb03c4af03 Mon Sep 17 00:00:00 2001 From: Chad Rosier Date: Tue, 18 Nov 2014 20:21:54 +0000 Subject: [PATCH] [Reassociate] Rename local variable to not use same name as a member variable. NFC. llvm-svn: 222248 --- llvm/lib/Transforms/Scalar/Reassociate.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/llvm/lib/Transforms/Scalar/Reassociate.cpp b/llvm/lib/Transforms/Scalar/Reassociate.cpp index 04e7240..b4bc9d4 100644 --- a/llvm/lib/Transforms/Scalar/Reassociate.cpp +++ b/llvm/lib/Transforms/Scalar/Reassociate.cpp @@ -586,7 +586,7 @@ static bool LinearizeExprTree(BinaryOperator *I, // ways to get to it. SmallVector, 8> Worklist; // (Op, Weight) Worklist.push_back(std::make_pair(I, APInt(Bitwidth, 1))); - bool MadeChange = false; + bool Changed = false; // Leaves of the expression are values that either aren't the right kind of // operation (eg: a constant, or a multiply in an add tree), or are, but have @@ -655,7 +655,7 @@ static bool LinearizeExprTree(BinaryOperator *I, // exactly one such use, drop this new use of the leaf. assert(!Op->hasOneUse() && "Only one use, but we got here twice!"); I->setOperand(OpIdx, UndefValue::get(I->getType())); - MadeChange = true; + Changed = true; // If the leaf is a binary operation of the right kind and we now see // that its multiple original uses were in fact all by nodes belonging @@ -699,7 +699,7 @@ static bool LinearizeExprTree(BinaryOperator *I, BO = LowerNegateToMultiply(BO); DEBUG(dbgs() << *BO << '\n'); Worklist.push_back(std::make_pair(BO, Weight)); - MadeChange = true; + Changed = true; continue; } @@ -739,7 +739,7 @@ static bool LinearizeExprTree(BinaryOperator *I, Ops.push_back(std::make_pair(Identity, APInt(Bitwidth, 1))); } - return MadeChange; + return Changed; } // RewriteExprTree - Now that the operands for this expression tree are -- 2.7.4