diff options
author | Owen Anderson <resistor@mac.com> | 2010-09-27 22:58:54 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2010-09-27 22:58:54 +0000 |
commit | 8d9e92ce2d848f74f0610ce783dad1a9256c4ffb (patch) | |
tree | cd69e9d89ed4c8207c8c739f902f4925eb3912d8 /lib/Transforms/Scalar/LoopUnrollPass.cpp | |
parent | 385cc5eede9f81717b03121baf47b7aa185b5128 (diff) | |
download | llvm-8d9e92ce2d848f74f0610ce783dad1a9256c4ffb.tar.gz llvm-8d9e92ce2d848f74f0610ce783dad1a9256c4ffb.tar.bz2 llvm-8d9e92ce2d848f74f0610ce783dad1a9256c4ffb.tar.xz |
Weight loop unrolling counts by nesting depth. Unrolling deeply nested loops tends to cause high
register pressure and thus excess spills, which we don't currently recover from well. This should
be re-evaluated in the future if our ability to generate good spills/splits improves.
Partial fix for <rdar://problem/7635585>.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@114919 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/LoopUnrollPass.cpp')
-rw-r--r-- | lib/Transforms/Scalar/LoopUnrollPass.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/LoopUnrollPass.cpp b/lib/Transforms/Scalar/LoopUnrollPass.cpp index 7da2b52da5..99a38b1ed0 100644 --- a/lib/Transforms/Scalar/LoopUnrollPass.cpp +++ b/lib/Transforms/Scalar/LoopUnrollPass.cpp @@ -158,7 +158,12 @@ bool LoopUnroll::runOnLoop(Loop *L, LPPassManager &LPM) { DEBUG(dbgs() << " Not unrolling loop with inlinable calls.\n"); return false; } - uint64_t Size = (uint64_t)LoopSize*Count; + + // NOTE: We multiply by the loop depth because unrolling inner loops of + // very deep nests tends to result in high register pressure, which we don't + // currently recover from very well. When and if the register allocator/ + // spiller improves to compensate, this should be re-evaluated. + uint64_t Size = (uint64_t)LoopSize*Count*L->getLoopDepth(); if (TripCount != 1 && Size > CurrentThreshold) { DEBUG(dbgs() << " Too large to fully unroll with count: " << Count << " because size: " << Size << ">" << CurrentThreshold << "\n"); |