OSDN Git Service

TailDuplicator: Remove no-op analyzeBranch call
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 7 Jun 2019 13:33:34 +0000 (13:33 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 7 Jun 2019 13:33:34 +0000 (13:33 +0000)
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.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@362800 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/TailDuplicator.cpp

index 15cbff4..4d3e02e 100644 (file)
@@ -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.