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 | 7a8db3a41a5c3c94ddd3f8cc7be0f464463e46de (patch) | |
| tree | 1f5d52b49337027aae89b043a8605ef5010027f1 /llvm/lib/Transforms/Scalar/LoopUnswitch.cpp | |
| parent | 54a4b840122ae09fe18b8dd52b55564e3776fe15 (diff) | |
| download | llvm-7a8db3a41a5c3c94ddd3f8cc7be0f464463e46de.zip llvm-7a8db3a41a5c3c94ddd3f8cc7be0f464463e46de.tar.gz llvm-7a8db3a41a5c3c94ddd3f8cc7be0f464463e46de.tar.bz2 | |
if CostMetrics says to never duplicate some code, don't unswitch a loop.
This prevents unswitching from duplicating indbr's.
llvm-svn: 85705
Diffstat (limited to 'llvm/lib/Transforms/Scalar/LoopUnswitch.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/LoopUnswitch.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp b/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp index 223d2b9..c7b00da 100644 --- a/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/llvm/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"); |