From 0e7fd2f1a84b2379a3b1df714fcc4d859de1a8da Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 18 Feb 2008 02:18:25 +0000 Subject: simplify some code, BreakUpSubtract always returns nonnull now. llvm-svn: 47251 --- llvm/lib/Transforms/Scalar/Reassociate.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'llvm/lib/Transforms/Scalar/Reassociate.cpp') diff --git a/llvm/lib/Transforms/Scalar/Reassociate.cpp b/llvm/lib/Transforms/Scalar/Reassociate.cpp index 3e2d38f76ba..47c767feb63 100644 --- a/llvm/lib/Transforms/Scalar/Reassociate.cpp +++ b/llvm/lib/Transforms/Scalar/Reassociate.cpp @@ -779,10 +779,8 @@ void Reassociate::ReassociateBB(BasicBlock *BB) { // see if we can convert it to X+-Y. if (BI->getOpcode() == Instruction::Sub) { if (ShouldBreakUpSubtract(BI)) { - if (Instruction *NI = BreakUpSubtract(BI)) { - MadeChange = true; - BI = NI; - } + BI = BreakUpSubtract(BI); + MadeChange = true; } else if (BinaryOperator::isNeg(BI)) { // Otherwise, this is a negation. See if the operand is a multiply tree // and if this is not an inner node of a multiply tree. -- cgit v1.2.3