[llvm-commits] [llvm] r93008 - /llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp
Duncan Sands
baldrick at free.fr
Fri Jan 8 09:51:48 PST 2010
Author: baldrick
Date: Fri Jan 8 11:51:48 2010
New Revision: 93008
URL: http://llvm.org/viewvc/llvm-project?rev=93008&view=rev
Log:
Suppress an unused variable warning when assertions are off;
remove some trailing whitespace while there.
Modified:
llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp?rev=93008&r1=93007&r2=93008&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp Fri Jan 8 11:51:48 2010
@@ -801,14 +801,15 @@
// No need for extra uses anymore.
delete DummyInst;
-
+
unsigned NumAddedValues = NewMulOps.size();
Value *V = EmitAddTreeOfValues(I, NewMulOps);
-
+
// Now that we have inserted the add tree, optimize it. This allows us to
// handle cases that require multiple factoring steps, such as this:
// A*A*B + A*A*C --> A*(A*B+A*C) --> A*(A*(B+C))
assert(NumAddedValues > 1 && "Each occurrence should contribute a value");
+ (void)NumAddedValues;
V = ReassociateExpression(cast<BinaryOperator>(V));
// Create the multiply.
More information about the llvm-commits
mailing list