diff options
author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2019-06-07 13:33:34 +0000 |
---|---|---|
committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2019-06-07 13:33:34 +0000 |
commit | 94a609e343f6160f58e9fda53e62a39b56dda7c0 (patch) | |
tree | 7dc55625647ff9a65e9a7d44a022e71b971b9339 /llvm/lib/CodeGen/TailDuplicator.cpp | |
parent | b2e96169b0a46abb8dc95f3255557b93d92ec36e (diff) | |
download | bcm5719-llvm-94a609e343f6160f58e9fda53e62a39b56dda7c0.tar.gz bcm5719-llvm-94a609e343f6160f58e9fda53e62a39b56dda7c0.zip |
TailDuplicator: Remove no-op analyzeBranch call
This could fail, which looked concerning. However nothing was actually
using the results of this. I assume this was intended to use the
anti-feature of analyzeBranch of removing instructions, but wasn't
actually calling it with AllowModify = true.
Fixes bug 42162.
llvm-svn: 362800
Diffstat (limited to 'llvm/lib/CodeGen/TailDuplicator.cpp')
-rw-r--r-- | llvm/lib/CodeGen/TailDuplicator.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/llvm/lib/CodeGen/TailDuplicator.cpp b/llvm/lib/CodeGen/TailDuplicator.cpp index 15cbff4fbcf..4d3e02e3174 100644 --- a/llvm/lib/CodeGen/TailDuplicator.cpp +++ b/llvm/lib/CodeGen/TailDuplicator.cpp @@ -856,11 +856,6 @@ bool TailDuplicator::tailDuplicate(bool IsSimple, MachineBasicBlock *TailBB, } appendCopies(PredBB, CopyInfos, Copies); - // Simplify - MachineBasicBlock *PredTBB = nullptr, *PredFBB = nullptr; - SmallVector<MachineOperand, 4> PredCond; - TII->analyzeBranch(*PredBB, PredTBB, PredFBB, PredCond); - NumTailDupAdded += TailBB->size() - 1; // subtract one for removed branch // Update the CFG. |