diff options
| author | Anna Thomas <anna@azul.com> | 2018-01-02 16:25:50 +0000 |
|---|---|---|
| committer | Anna Thomas <anna@azul.com> | 2018-01-02 16:25:50 +0000 |
| commit | bdb94309177f8146f458e2cb8bd6224c2d8f295e (patch) | |
| tree | 0633321a888705e171b9bb66556e9a319119c692 /llvm/lib/Target/XCore/XCoreTargetMachine.cpp | |
| parent | 29e185696674539a8ad1d4b1d65a5419b8b2c777 (diff) | |
| download | bcm5719-llvm-bdb94309177f8146f458e2cb8bd6224c2d8f295e.tar.gz bcm5719-llvm-bdb94309177f8146f458e2cb8bd6224c2d8f295e.zip | |
[BasicBlockUtils] Check for unreachable preds before updating LI in UpdateAnalysisInformation
Summary:
We are incorrectly updating the LI when loop-simplify generates
dedicated exit blocks for a loop. The issue is that there's an implicit
assumption that the Preds passed into UpdateAnalysisInformation are
reachable. However, this is not true and breaks LI by incorrectly
updating the header of a loop.
One such case is when we generate dedicated exits when the exit block is
a landing pad (through SplitLandingPadPredecessors). There maybe other
cases as well, since we do not guarantee that Preds passed in are
reachable basic blocks.
The added test case shows how loop-simplify breaks LI for the outer loop (and DT in turn)
after we try to generate the LoopSimplifyForm.
Reviewers: davide, chandlerc, sanjoy
Reviewed By: davide
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D41519
llvm-svn: 321653
Diffstat (limited to 'llvm/lib/Target/XCore/XCoreTargetMachine.cpp')
0 files changed, 0 insertions, 0 deletions

