[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