6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-03-06 20:33:54 +00:00
Code Issues Projects Releases Wiki Activity

Use IRBuilder while simplifying unconditional branch.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@131551 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Devang Patel 2011-05-18 18:28:48 +00:00
parent f60364de44
commit a23812cfbd
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/Transforms/Utils/SimplifyCFG.cpp
View File

@ -65,7 +65,7 @@ class SimplifyCFGOpt {
bool SimplifyUnreachable(UnreachableInst *UI); bool SimplifyUnreachable(UnreachableInst *UI);
bool SimplifySwitch(SwitchInst *SI); bool SimplifySwitch(SwitchInst *SI);
bool SimplifyIndirectBr(IndirectBrInst *IBI); bool SimplifyIndirectBr(IndirectBrInst *IBI);
bool SimplifyUncondBranch(BranchInst *BI); bool SimplifyUncondBranch(BranchInst *BI, IRBuilder <> &Builder);
bool SimplifyCondBranch(BranchInst *BI); bool SimplifyCondBranch(BranchInst *BI);
public: public:
@ -1923,8 +1923,10 @@ static bool SimplifyIndirectBrOnSelect(IndirectBrInst *IBI, SelectInst *SI) {
/// We prefer to split the edge to 'end' so that there is a true/false entry to /// We prefer to split the edge to 'end' so that there is a true/false entry to
/// the PHI, merging the third icmp into the switch. /// the PHI, merging the third icmp into the switch.
static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI, static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI,
const TargetData *TD) { const TargetData *TD,
IRBuilder<> &Builder) {
BasicBlock *BB = ICI->getParent(); BasicBlock *BB = ICI->getParent();
// If the block has any PHIs in it or the icmp has multiple uses, it is too // If the block has any PHIs in it or the icmp has multiple uses, it is too
// complex. // complex.
if (isa<PHINode>(BB->begin()) || !ICI->hasOneUse()) return false; if (isa<PHINode>(BB->begin()) || !ICI->hasOneUse()) return false;
@ -2002,7 +2004,9 @@ static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI,
SI->addCase(Cst, NewBB); SI->addCase(Cst, NewBB);
// NewBB branches to the phi block, add the uncond branch and the phi entry. // NewBB branches to the phi block, add the uncond branch and the phi entry.
BranchInst::Create(SuccBlock, NewBB); Builder.SetInsertPoint(NewBB);
Builder.SetCurrentDebugLocation(SI->getDebugLoc());
Builder.CreateBr(SuccBlock);
PHIUse->addIncoming(NewCst, NewBB); PHIUse->addIncoming(NewCst, NewBB);
return true; return true;
} }
@ -2503,7 +2507,7 @@ bool SimplifyCFGOpt::SimplifyIndirectBr(IndirectBrInst *IBI) {
return Changed; return Changed;
} }
bool SimplifyCFGOpt::SimplifyUncondBranch(BranchInst *BI) { bool SimplifyCFGOpt::SimplifyUncondBranch(BranchInst *BI, IRBuilder<> &Builder){
BasicBlock *BB = BI->getParent(); BasicBlock *BB = BI->getParent();
// If the Terminator is the only non-phi instruction, simplify the block. // If the Terminator is the only non-phi instruction, simplify the block.
@ -2518,7 +2522,8 @@ bool SimplifyCFGOpt::SimplifyUncondBranch(BranchInst *BI) {
if (ICI->isEquality() && isa<ConstantInt>(ICI->getOperand(1))) { if (ICI->isEquality() && isa<ConstantInt>(ICI->getOperand(1))) {
for (++I; isa<DbgInfoIntrinsic>(I); ++I) for (++I; isa<DbgInfoIntrinsic>(I); ++I)
; ;
if (I->isTerminator() && TryToSimplifyUncondBranchWithICmpInIt(ICI, TD)) if (I->isTerminator()
&& TryToSimplifyUncondBranchWithICmpInIt(ICI, TD, Builder))
return true; return true;
} }
@ -2651,7 +2656,7 @@ bool SimplifyCFGOpt::run(BasicBlock *BB) {
if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator())) { if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator())) {
if (BI->isUnconditional()) { if (BI->isUnconditional()) {
if (SimplifyUncondBranch(BI)) return true; if (SimplifyUncondBranch(BI, Builder)) return true;
} else { } else {
if (SimplifyCondBranch(BI)) return true; if (SimplifyCondBranch(BI)) return true;
} }