[LPM] Fix PR18616 where the shifts to the loop pass manager to extract
authorChandler Carruth <chandlerc@gmail.com>
Tue, 28 Jan 2014 01:25:38 +0000 (01:25 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Tue, 28 Jan 2014 01:25:38 +0000 (01:25 +0000)
commit5ea4a545518b5fe1dbd1decc6435c8de35ec9ed1
tree8945de0504b9bfdbbb99cf7f220f702bd866fd85
parentefbb39740c32b6b8311d27b0f279a7d523fbdf2d
[LPM] Fix PR18616 where the shifts to the loop pass manager to extract
LCSSA from it caused a crasher with the LoopUnroll pass.

This crasher is really nasty. We destroy LCSSA form in a suprising way.
When unrolling a loop into an outer loop, we not only need to restore
LCSSA form for the outer loop, but for all children of the outer loop.
This is somewhat obvious in retrospect, but hey!

While this seems pretty heavy-handed, it's not that bad. Fundamentally,
we only do this when we unroll a loop, which is already a heavyweight
operation. We're unrolling all of these hypothetical inner loops as
well, so their size and complexity is already on the critical path. This
is just adding another pass over them to re-canonicalize.

I have a test case from PR18616 that is great for reproducing this, but
pretty useless to check in as it relies on many 10s of nested empty
loops that get unrolled and deleted in just the right order. =/ What's
worse is that investigating this has exposed another source of failure
that is likely to be even harder to test. I'll try to come up with test
cases for these fixes, but I want to get the fixes into the tree first
as they're causing crashes in the wild.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@200273 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/Transforms/Utils/LoopUtils.h
lib/Transforms/Utils/LCSSA.cpp
lib/Transforms/Utils/LoopUnroll.cpp