Skip to content

Commit

Permalink
Implement Jakob's suggestion on how to detect fall thought without ca…
Browse files Browse the repository at this point in the history
…lling

AnalyzeBranch.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@132981 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
espindola committed Jun 14, 2011
1 parent 79abd1c commit f5b5c51
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 14 deletions.
31 changes: 19 additions & 12 deletions lib/CodeGen/AsmPrinter/AsmPrinter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1934,19 +1934,26 @@ isBlockOnlyReachableByFallthrough(const MachineBasicBlock *MBB) const {
if (Pred->empty())
return true;

// Otherwise, ask the backend.
const TargetInstrInfo *TII = MF->getTarget().getInstrInfo();
MachineBasicBlock *PredTBB = NULL, *PredFBB = NULL;
SmallVector<MachineOperand, 4> PredCond;
if (TII->AnalyzeBranch(*Pred, PredTBB, PredFBB, PredCond))
return false;

if (PredTBB == MBB || PredFBB == MBB)
return false;
// Check the terminators in the previous blocks
for (MachineBasicBlock::iterator II = Pred->getFirstTerminator(),
IE = Pred->end(); II != IE; ++II) {
MachineInstr &MI = *II;

// If it is not a simple branch, we are in a table somewhere.
if (!MI.getDesc().isBranch() || MI.getDesc().isIndirectBranch())
return false;

// If we are the operands of one of the branches, this is not
// a fall through.
for (MachineInstr::mop_iterator OI = MI.operands_begin(),
OE = MI.operands_end(); OI != OE; ++OI) {
const MachineOperand& OP = *OI;
if (OP.isMBB() && OP.getMBB() == MBB)
return false;
}
}

// This is a fall through if there is no conditions in the bb
// or if there is no explicit false branch.
return PredCond.empty() || !PredFBB;
return true;
}


Expand Down
4 changes: 2 additions & 2 deletions test/CodeGen/Thumb2/machine-licm.ll
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,15 @@ entry:
br i1 %0, label %return, label %bb.nph

bb.nph: ; preds = %entry
; CHECK: LBB0_1:
; CHECK: BB#1
; CHECK: movw r[[R2:[0-9]+]], :lower16:L_GV$non_lazy_ptr
; CHECK: movt r[[R2]], :upper16:L_GV$non_lazy_ptr
; CHECK: ldr{{(.w)?}} r[[R2b:[0-9]+]], [r[[R2]]
; CHECK: ldr{{.*}}, [r[[R2b]]
; CHECK: LBB0_2
; CHECK-NOT: LCPI0_0:

; PIC: LBB0_1:
; PIC: BB#1
; PIC: movw r[[R2:[0-9]+]], :lower16:(L_GV$non_lazy_ptr-(LPC0_0+4))
; PIC: movt r[[R2]], :upper16:(L_GV$non_lazy_ptr-(LPC0_0+4))
; PIC: add r[[R2]], pc
Expand Down

0 comments on commit f5b5c51

Please sign in to comment.