diff options
| author | Owen Anderson <resistor@mac.com> | 2010-09-09 20:02:23 +0000 |
|---|---|---|
| committer | Owen Anderson <resistor@mac.com> | 2010-09-09 20:02:23 +0000 |
| commit | 547462b98cc0da6505ffafb1d47f0ed6f7518288 (patch) | |
| tree | 566b2cf8432874ababcba8e10e40233913dc004b /lib/Transforms | |
| parent | cc443c37eed4fca3ab592bb7bf16a3bf6ec6c510 (diff) | |
| download | external_llvm-547462b98cc0da6505ffafb1d47f0ed6f7518288.zip external_llvm-547462b98cc0da6505ffafb1d47f0ed6f7518288.tar.gz external_llvm-547462b98cc0da6505ffafb1d47f0ed6f7518288.tar.bz2 | |
Revert r113439, which relaxed the requirement that loops containing calls cannot be unrolled. After some discussion,
there seems to be a better way to achieve the same effect.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@113528 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/LoopUnrollPass.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/Transforms/Scalar/LoopUnrollPass.cpp b/lib/Transforms/Scalar/LoopUnrollPass.cpp index 4d23178..7968939 100644 --- a/lib/Transforms/Scalar/LoopUnrollPass.cpp +++ b/lib/Transforms/Scalar/LoopUnrollPass.cpp @@ -155,10 +155,8 @@ bool LoopUnroll::runOnLoop(Loop *L, LPPassManager &LPM) { unsigned LoopSize = ApproximateLoopSize(L, NumCalls); DEBUG(dbgs() << " Loop Size = " << LoopSize << "\n"); if (NumCalls != 0) { - // Even for a loop that contains calls, it can still be profitable to - // unroll if the loop is really, REALLY small. - DEBUG(dbgs() <<" Using lower threshold for loop with function calls.\n"); - CurrentThreshold = OptSizeUnrollThreshold; + DEBUG(dbgs() << " Not unrolling loop with function calls.\n"); + return false; } uint64_t Size = (uint64_t)LoopSize*Count; if (TripCount != 1 && Size > CurrentThreshold) { |