[llvm] r271615 - [LoopUnroll] Set correct thresholds for new recently enabled unrolling heuristic.

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 17:16:47 PDT 2016


Author: mzolotukhin
Date: Thu Jun  2 19:16:46 2016
New Revision: 271615

URL: http://llvm.org/viewvc/llvm-project?rev=271615&view=rev
Log:
[LoopUnroll] Set correct thresholds for new recently enabled unrolling heuristic.

In r270478, where I enabled the new heuristic I posted testing results,
which I got when explicitly passed the thresholds values via CL options.
However, setting the CL options init-values is not enough to change the
default values of thresholds, so I'm changing them in another place now.

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

Modified: llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp?rev=271615&r1=271614&r2=271615&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp Thu Jun  2 19:16:46 2016
@@ -114,8 +114,8 @@ static TargetTransformInfo::UnrollingPre
 
   // Set up the defaults
   UP.Threshold = 150;
-  UP.PercentDynamicCostSavedThreshold = 20;
-  UP.DynamicCostSavingsDiscount = 2000;
+  UP.PercentDynamicCostSavedThreshold = 50;
+  UP.DynamicCostSavingsDiscount = 100;
   UP.OptSizeThreshold = 0;
   UP.PartialThreshold = UP.Threshold;
   UP.PartialOptSizeThreshold = 0;




More information about the llvm-commits mailing list