[llvm] r288154 - [CVP] Remove cvp-dont-process-adds flag

Artur Pilipenko via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 08:24:57 PST 2016


Author: apilipenko
Date: Tue Nov 29 10:24:57 2016
New Revision: 288154

URL: http://llvm.org/viewvc/llvm-project?rev=288154&view=rev
Log:
[CVP] Remove cvp-dont-process-adds flag

The flag was introduced because the optimization controlled by the flag initially caused regressions. All the regressions were fixed some time ago and the flag has been false for quite a while. 

Modified:
    llvm/trunk/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp?rev=288154&r1=288153&r2=288154&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp Tue Nov 29 10:24:57 2016
@@ -41,8 +41,6 @@ STATISTIC(NumSDivs,     "Number of sdiv
 STATISTIC(NumAShrs,     "Number of ashr converted to lshr");
 STATISTIC(NumSRems,     "Number of srem converted to urem");
 
-static cl::opt<bool> DontProcessAdds("cvp-dont-process-adds", cl::init(false));
-
 namespace {
   class CorrelatedValuePropagation : public FunctionPass {
   public:
@@ -407,9 +405,6 @@ static bool processAShr(BinaryOperator *
 static bool processAdd(BinaryOperator *AddOp, LazyValueInfo *LVI) {
   typedef OverflowingBinaryOperator OBO;
 
-  if (DontProcessAdds)
-    return false;
-
   if (AddOp->getType()->isVectorTy() || hasLocalDefs(AddOp))
     return false;
 




More information about the llvm-commits mailing list