diff options
| author | Simon Dardis <simon.dardis@mips.com> | 2017-10-26 10:58:36 +0000 |
|---|---|---|
| committer | Simon Dardis <simon.dardis@mips.com> | 2017-10-26 10:58:36 +0000 |
| commit | 13452383cd48b973da4fd31c8ebdf3c313438f0e (patch) | |
| tree | 4cab9dbb602fe87ca5f7c74ca677225ca2d84e8d /llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp | |
| parent | 826b7832140bda013140d6bcb370331f64f85823 (diff) | |
| download | bcm5719-llvm-13452383cd48b973da4fd31c8ebdf3c313438f0e.tar.gz bcm5719-llvm-13452383cd48b973da4fd31c8ebdf3c313438f0e.zip | |
[mips] Fix PR35071
PR35071 exposed the fact that MipsInstrInfo::removeBranch did not walk past
debug instructions when removing branches for the control flow optimizer, which
lead to duplicated conditional branches. If the target of the branch was a
removable block, only the conditional branch in the terminating position would
have it's MBB operands updated, leaving the first branch with a dangling MBB
operand. The MIPS long branch pass would then trigger an assertion when
attempting to examine the instruction with dangling MBB operand.
This resolves PR35071.
Thanks to Alex Richardson for reporting the issue!
Reviewers: atanasyan
Differential Revision: https://reviews.llvm.org/D39288
llvm-svn: 316654
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp')
0 files changed, 0 insertions, 0 deletions
