diff options
author | Chris Lattner <sabre@nondot.org> | 2009-11-01 03:42:55 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2009-11-01 03:42:55 +0000 |
commit | da9c281121537d9823ddeecf7bb15095d95b2722 (patch) | |
tree | 6e8835d1018877642335241f8c84ad2b02825269 | |
parent | 0a4c6789d5adafb6eb33080fe1833b416a152d7c (diff) | |
download | external_llvm-da9c281121537d9823ddeecf7bb15095d95b2722.zip external_llvm-da9c281121537d9823ddeecf7bb15095d95b2722.tar.gz external_llvm-da9c281121537d9823ddeecf7bb15095d95b2722.tar.bz2 |
if CostMetrics says to never duplicate some code, don't unswitch a loop.
This prevents unswitching from duplicating indbr's.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@85705 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/LoopUnswitch.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/LoopUnswitch.cpp b/lib/Transforms/Scalar/LoopUnswitch.cpp index 223d2b9..c7b00da 100644 --- a/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/lib/Transforms/Scalar/LoopUnswitch.cpp @@ -430,7 +430,8 @@ bool LoopUnswitch::UnswitchIfProfitable(Value *LoopCond, Constant *Val){ // large numbers of branches which cause loop unswitching to go crazy. // This is a very ad-hoc heuristic. if (Metrics.NumInsts > Threshold || - Metrics.NumBlocks * 5 > Threshold) { + Metrics.NumBlocks * 5 > Threshold || + Metrics.NeverInline) { DEBUG(errs() << "NOT unswitching loop %" << currentLoop->getHeader()->getName() << ", cost too high: " << currentLoop->getBlocks().size() << "\n"); |