2002-05-21 20:50:24 +00:00
|
|
|
//===- SimplifyCFG.cpp - Code to perform CFG simplification ---------------===//
|
2005-04-21 23:48:37 +00:00
|
|
|
//
|
2003-10-20 19:43:21 +00:00
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-29 20:36:04 +00:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2005-04-21 23:48:37 +00:00
|
|
|
//
|
2003-10-20 19:43:21 +00:00
|
|
|
//===----------------------------------------------------------------------===//
|
2002-05-21 20:50:24 +00:00
|
|
|
//
|
2002-10-08 21:36:33 +00:00
|
|
|
// Peephole optimize the CFG.
|
2002-05-21 20:50:24 +00:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2004-06-20 01:13:18 +00:00
|
|
|
#define DEBUG_TYPE "simplifycfg"
|
2002-05-21 20:50:24 +00:00
|
|
|
#include "llvm/Transforms/Utils/Local.h"
|
2004-02-11 03:36:04 +00:00
|
|
|
#include "llvm/Constants.h"
|
|
|
|
#include "llvm/Instructions.h"
|
2009-02-03 22:12:02 +00:00
|
|
|
#include "llvm/IntrinsicInst.h"
|
2004-02-24 05:38:11 +00:00
|
|
|
#include "llvm/Type.h"
|
2007-01-19 21:13:56 +00:00
|
|
|
#include "llvm/DerivedTypes.h"
|
2009-05-13 18:25:07 +00:00
|
|
|
#include "llvm/GlobalVariable.h"
|
2010-12-14 06:17:25 +00:00
|
|
|
#include "llvm/Analysis/InstructionSimplify.h"
|
2010-02-05 22:03:18 +00:00
|
|
|
#include "llvm/Target/TargetData.h"
|
2005-09-19 23:49:37 +00:00
|
|
|
#include "llvm/Transforms/Utils/BasicBlockUtils.h"
|
2009-10-30 22:39:04 +00:00
|
|
|
#include "llvm/ADT/DenseMap.h"
|
2007-02-13 02:10:56 +00:00
|
|
|
#include "llvm/ADT/SmallVector.h"
|
2007-04-02 01:44:59 +00:00
|
|
|
#include "llvm/ADT/SmallPtrSet.h"
|
2008-06-12 21:15:59 +00:00
|
|
|
#include "llvm/ADT/Statistic.h"
|
2010-12-13 02:00:58 +00:00
|
|
|
#include "llvm/ADT/STLExtras.h"
|
2010-12-14 06:17:25 +00:00
|
|
|
#include "llvm/Support/CFG.h"
|
2011-01-29 04:46:23 +00:00
|
|
|
#include "llvm/Support/CommandLine.h"
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
#include "llvm/Support/ConstantRange.h"
|
2010-12-14 06:17:25 +00:00
|
|
|
#include "llvm/Support/Debug.h"
|
|
|
|
#include "llvm/Support/raw_ostream.h"
|
2002-05-21 20:50:24 +00:00
|
|
|
#include <algorithm>
|
2004-02-24 07:23:58 +00:00
|
|
|
#include <set>
|
2004-10-18 04:07:22 +00:00
|
|
|
#include <map>
|
2004-01-09 06:12:26 +00:00
|
|
|
using namespace llvm;
|
2003-11-11 22:41:34 +00:00
|
|
|
|
2011-01-29 04:46:23 +00:00
|
|
|
static cl::opt<bool>
|
|
|
|
DupRet("simplifycfg-dup-ret", cl::Hidden, cl::init(false),
|
|
|
|
cl::desc("Duplicate return instructions into unconditional branches"));
|
|
|
|
|
2008-06-12 21:15:59 +00:00
|
|
|
STATISTIC(NumSpeculations, "Number of speculative executed instructions");
|
|
|
|
|
2010-02-05 22:03:18 +00:00
|
|
|
namespace {
|
|
|
|
class SimplifyCFGOpt {
|
|
|
|
const TargetData *const TD;
|
|
|
|
|
|
|
|
Value *isValueEqualityComparison(TerminatorInst *TI);
|
|
|
|
BasicBlock *GetValueEqualityComparisonCases(TerminatorInst *TI,
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > &Cases);
|
|
|
|
bool SimplifyEqualityComparisonWithOnlyPredecessor(TerminatorInst *TI,
|
|
|
|
BasicBlock *Pred);
|
|
|
|
bool FoldValueComparisonIntoPredecessors(TerminatorInst *TI);
|
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyReturn(ReturnInst *RI);
|
|
|
|
bool SimplifyUnwind(UnwindInst *UI);
|
|
|
|
bool SimplifyUnreachable(UnreachableInst *UI);
|
|
|
|
bool SimplifySwitch(SwitchInst *SI);
|
|
|
|
bool SimplifyIndirectBr(IndirectBrInst *IBI);
|
|
|
|
bool SimplifyUncondBranch(BranchInst *BI);
|
|
|
|
bool SimplifyCondBranch(BranchInst *BI);
|
|
|
|
|
2010-02-05 22:03:18 +00:00
|
|
|
public:
|
|
|
|
explicit SimplifyCFGOpt(const TargetData *td) : TD(td) {}
|
|
|
|
bool run(BasicBlock *BB);
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2005-08-03 00:19:45 +00:00
|
|
|
/// SafeToMergeTerminators - Return true if it is safe to merge these two
|
|
|
|
/// terminator instructions together.
|
|
|
|
///
|
|
|
|
static bool SafeToMergeTerminators(TerminatorInst *SI1, TerminatorInst *SI2) {
|
|
|
|
if (SI1 == SI2) return false; // Can't merge with self!
|
|
|
|
|
|
|
|
// It is not safe to merge these two switch instructions if they have a common
|
|
|
|
// successor, and if that successor has a PHI node, and if *that* PHI node has
|
|
|
|
// conflicting incoming values from the two switch blocks.
|
|
|
|
BasicBlock *SI1BB = SI1->getParent();
|
|
|
|
BasicBlock *SI2BB = SI2->getParent();
|
2007-04-02 01:44:59 +00:00
|
|
|
SmallPtrSet<BasicBlock*, 16> SI1Succs(succ_begin(SI1BB), succ_end(SI1BB));
|
2005-08-03 00:19:45 +00:00
|
|
|
|
|
|
|
for (succ_iterator I = succ_begin(SI2BB), E = succ_end(SI2BB); I != E; ++I)
|
|
|
|
if (SI1Succs.count(*I))
|
|
|
|
for (BasicBlock::iterator BBI = (*I)->begin();
|
|
|
|
isa<PHINode>(BBI); ++BBI) {
|
|
|
|
PHINode *PN = cast<PHINode>(BBI);
|
|
|
|
if (PN->getIncomingValueForBlock(SI1BB) !=
|
|
|
|
PN->getIncomingValueForBlock(SI2BB))
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// AddPredecessorToBlock - Update PHI nodes in Succ to indicate that there will
|
|
|
|
/// now be entries in it from the 'NewPred' block. The values that will be
|
|
|
|
/// flowing into the PHI nodes will be the same as those coming in from
|
|
|
|
/// ExistPred, an existing predecessor of Succ.
|
|
|
|
static void AddPredecessorToBlock(BasicBlock *Succ, BasicBlock *NewPred,
|
|
|
|
BasicBlock *ExistPred) {
|
|
|
|
if (!isa<PHINode>(Succ->begin())) return; // Quick exit if nothing to do
|
|
|
|
|
2008-07-13 22:23:11 +00:00
|
|
|
PHINode *PN;
|
|
|
|
for (BasicBlock::iterator I = Succ->begin();
|
|
|
|
(PN = dyn_cast<PHINode>(I)); ++I)
|
|
|
|
PN->addIncoming(PN->getIncomingValueForBlock(ExistPred), NewPred);
|
2005-08-03 00:19:45 +00:00
|
|
|
}
|
|
|
|
|
2005-08-03 00:11:16 +00:00
|
|
|
|
2010-12-14 07:00:00 +00:00
|
|
|
/// GetIfCondition - Given a basic block (BB) with two predecessors (and at
|
|
|
|
/// least one PHI node in it), check to see if the merge at this block is due
|
2004-02-11 03:36:04 +00:00
|
|
|
/// to an "if condition". If so, return the boolean condition that determines
|
|
|
|
/// which entry into BB will be taken. Also, return by references the block
|
|
|
|
/// that will be entered from if the condition is true, and the block that will
|
|
|
|
/// be entered if the condition is false.
|
2005-04-21 23:48:37 +00:00
|
|
|
///
|
2010-12-14 07:15:21 +00:00
|
|
|
/// This does no checking to see if the true/false blocks have large or unsavory
|
|
|
|
/// instructions in them.
|
2010-12-14 07:00:00 +00:00
|
|
|
static Value *GetIfCondition(BasicBlock *BB, BasicBlock *&IfTrue,
|
|
|
|
BasicBlock *&IfFalse) {
|
|
|
|
PHINode *SomePHI = cast<PHINode>(BB->begin());
|
|
|
|
assert(SomePHI->getNumIncomingValues() == 2 &&
|
2004-02-11 03:36:04 +00:00
|
|
|
"Function can only handle blocks with 2 predecessors!");
|
2010-12-14 07:00:00 +00:00
|
|
|
BasicBlock *Pred1 = SomePHI->getIncomingBlock(0);
|
|
|
|
BasicBlock *Pred2 = SomePHI->getIncomingBlock(1);
|
2004-02-11 03:36:04 +00:00
|
|
|
|
|
|
|
// We can only handle branches. Other control flow will be lowered to
|
|
|
|
// branches if possible anyway.
|
2010-12-14 07:15:21 +00:00
|
|
|
BranchInst *Pred1Br = dyn_cast<BranchInst>(Pred1->getTerminator());
|
|
|
|
BranchInst *Pred2Br = dyn_cast<BranchInst>(Pred2->getTerminator());
|
|
|
|
if (Pred1Br == 0 || Pred2Br == 0)
|
2004-02-11 03:36:04 +00:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
// Eliminate code duplication by ensuring that Pred1Br is conditional if
|
|
|
|
// either are.
|
|
|
|
if (Pred2Br->isConditional()) {
|
|
|
|
// If both branches are conditional, we don't have an "if statement". In
|
|
|
|
// reality, we could transform this case, but since the condition will be
|
|
|
|
// required anyway, we stand no chance of eliminating it, so the xform is
|
|
|
|
// probably not profitable.
|
|
|
|
if (Pred1Br->isConditional())
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
std::swap(Pred1, Pred2);
|
|
|
|
std::swap(Pred1Br, Pred2Br);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (Pred1Br->isConditional()) {
|
2010-12-14 07:15:21 +00:00
|
|
|
// The only thing we have to watch out for here is to make sure that Pred2
|
|
|
|
// doesn't have incoming edges from other blocks. If it does, the condition
|
|
|
|
// doesn't dominate BB.
|
|
|
|
if (Pred2->getSinglePredecessor() == 0)
|
|
|
|
return 0;
|
|
|
|
|
2004-02-11 03:36:04 +00:00
|
|
|
// If we found a conditional branch predecessor, make sure that it branches
|
|
|
|
// to BB and Pred2Br. If it doesn't, this isn't an "if statement".
|
|
|
|
if (Pred1Br->getSuccessor(0) == BB &&
|
|
|
|
Pred1Br->getSuccessor(1) == Pred2) {
|
|
|
|
IfTrue = Pred1;
|
|
|
|
IfFalse = Pred2;
|
|
|
|
} else if (Pred1Br->getSuccessor(0) == Pred2 &&
|
|
|
|
Pred1Br->getSuccessor(1) == BB) {
|
|
|
|
IfTrue = Pred2;
|
|
|
|
IfFalse = Pred1;
|
|
|
|
} else {
|
|
|
|
// We know that one arm of the conditional goes to BB, so the other must
|
|
|
|
// go somewhere unrelated, and this must not be an "if statement".
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
return Pred1Br->getCondition();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Ok, if we got here, both predecessors end with an unconditional branch to
|
|
|
|
// BB. Don't panic! If both blocks only have a single (identical)
|
|
|
|
// predecessor, and THAT is a conditional branch, then we're all ok!
|
2010-12-14 07:15:21 +00:00
|
|
|
BasicBlock *CommonPred = Pred1->getSinglePredecessor();
|
|
|
|
if (CommonPred == 0 || CommonPred != Pred2->getSinglePredecessor())
|
2004-02-11 03:36:04 +00:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
// Otherwise, if this is a conditional branch, then we can use it!
|
2010-12-14 07:15:21 +00:00
|
|
|
BranchInst *BI = dyn_cast<BranchInst>(CommonPred->getTerminator());
|
|
|
|
if (BI == 0) return 0;
|
|
|
|
|
|
|
|
assert(BI->isConditional() && "Two successors but not conditional?");
|
|
|
|
if (BI->getSuccessor(0) == Pred1) {
|
|
|
|
IfTrue = Pred1;
|
|
|
|
IfFalse = Pred2;
|
|
|
|
} else {
|
|
|
|
IfTrue = Pred2;
|
|
|
|
IfFalse = Pred1;
|
2004-02-11 03:36:04 +00:00
|
|
|
}
|
2010-12-14 07:15:21 +00:00
|
|
|
return BI->getCondition();
|
2004-02-11 03:36:04 +00:00
|
|
|
}
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// DominatesMergePoint - If we have a merge point of an "if condition" as
|
|
|
|
/// accepted above, return true if the specified value dominates the block. We
|
|
|
|
/// don't handle the true generality of domination here, just a special case
|
|
|
|
/// which works well enough for us.
|
|
|
|
///
|
|
|
|
/// If AggressiveInsts is non-null, and if V does not dominate BB, we check to
|
|
|
|
/// see if V (which must be an instruction) is cheap to compute and is
|
|
|
|
/// non-trapping. If both are true, the instruction is inserted into the set
|
|
|
|
/// and true is returned.
|
2004-10-14 05:13:36 +00:00
|
|
|
static bool DominatesMergePoint(Value *V, BasicBlock *BB,
|
2010-12-14 07:41:39 +00:00
|
|
|
SmallPtrSet<Instruction*, 4> *AggressiveInsts) {
|
2004-04-09 22:50:22 +00:00
|
|
|
Instruction *I = dyn_cast<Instruction>(V);
|
2006-10-20 00:42:07 +00:00
|
|
|
if (!I) {
|
|
|
|
// Non-instructions all dominate instructions, but not all constantexprs
|
|
|
|
// can be executed unconditionally.
|
|
|
|
if (ConstantExpr *C = dyn_cast<ConstantExpr>(V))
|
|
|
|
if (C->canTrap())
|
|
|
|
return false;
|
|
|
|
return true;
|
|
|
|
}
|
2004-04-09 22:50:22 +00:00
|
|
|
BasicBlock *PBB = I->getParent();
|
|
|
|
|
2005-02-27 06:18:25 +00:00
|
|
|
// We don't want to allow weird loops that might have the "if condition" in
|
2004-04-09 22:50:22 +00:00
|
|
|
// the bottom of this block.
|
|
|
|
if (PBB == BB) return false;
|
|
|
|
|
|
|
|
// If this instruction is defined in a block that contains an unconditional
|
|
|
|
// branch to BB, then it must be in the 'conditional' part of the "if
|
2010-12-14 07:41:39 +00:00
|
|
|
// statement". If not, it definitely dominates the region.
|
|
|
|
BranchInst *BI = dyn_cast<BranchInst>(PBB->getTerminator());
|
|
|
|
if (BI == 0 || BI->isConditional() || BI->getSuccessor(0) != BB)
|
|
|
|
return true;
|
2009-07-17 04:28:42 +00:00
|
|
|
|
2010-12-14 07:41:39 +00:00
|
|
|
// If we aren't allowing aggressive promotion anymore, then don't consider
|
|
|
|
// instructions in the 'if region'.
|
|
|
|
if (AggressiveInsts == 0) return false;
|
|
|
|
|
|
|
|
// Okay, it looks like the instruction IS in the "condition". Check to
|
|
|
|
// see if it's a cheap instruction to unconditionally compute, and if it
|
|
|
|
// only uses stuff defined outside of the condition. If so, hoist it out.
|
|
|
|
if (!I->isSafeToSpeculativelyExecute())
|
|
|
|
return false;
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2010-12-14 07:41:39 +00:00
|
|
|
switch (I->getOpcode()) {
|
|
|
|
default: return false; // Cannot hoist this out safely.
|
|
|
|
case Instruction::Load:
|
|
|
|
// We have to check to make sure there are no instructions before the
|
|
|
|
// load in its basic block, as we are going to hoist the load out to its
|
|
|
|
// predecessor.
|
|
|
|
if (PBB->getFirstNonPHIOrDbg() != I)
|
|
|
|
return false;
|
|
|
|
break;
|
2011-02-25 10:33:33 +00:00
|
|
|
case Instruction::GetElementPtr:
|
|
|
|
// GEPs are cheap if all indices are constant.
|
|
|
|
if (!cast<GetElementPtrInst>(I)->hasAllConstantIndices())
|
2011-02-24 22:46:11 +00:00
|
|
|
return false;
|
|
|
|
break;
|
2010-12-14 07:41:39 +00:00
|
|
|
case Instruction::Add:
|
|
|
|
case Instruction::Sub:
|
|
|
|
case Instruction::And:
|
|
|
|
case Instruction::Or:
|
|
|
|
case Instruction::Xor:
|
|
|
|
case Instruction::Shl:
|
|
|
|
case Instruction::LShr:
|
|
|
|
case Instruction::AShr:
|
|
|
|
case Instruction::ICmp:
|
|
|
|
break; // These are all cheap and non-trapping instructions.
|
|
|
|
}
|
2004-02-11 03:36:04 +00:00
|
|
|
|
2010-12-14 07:41:39 +00:00
|
|
|
// Okay, we can only really hoist these out if their operands are not
|
|
|
|
// defined in the conditional region.
|
|
|
|
for (User::op_iterator i = I->op_begin(), e = I->op_end(); i != e; ++i)
|
|
|
|
if (!DominatesMergePoint(*i, BB, 0))
|
|
|
|
return false;
|
|
|
|
// Okay, it's safe to do this! Remember this instruction.
|
|
|
|
AggressiveInsts->insert(I);
|
2004-02-11 03:36:04 +00:00
|
|
|
return true;
|
|
|
|
}
|
2002-05-21 20:50:24 +00:00
|
|
|
|
2010-02-05 22:03:18 +00:00
|
|
|
/// GetConstantInt - Extract ConstantInt from value, looking through IntToPtr
|
|
|
|
/// and PointerNullValue. Return NULL if value is not a constant int.
|
2010-12-13 03:30:12 +00:00
|
|
|
static ConstantInt *GetConstantInt(Value *V, const TargetData *TD) {
|
2010-02-05 22:03:18 +00:00
|
|
|
// Normal constant int.
|
|
|
|
ConstantInt *CI = dyn_cast<ConstantInt>(V);
|
2010-02-16 11:11:14 +00:00
|
|
|
if (CI || !TD || !isa<Constant>(V) || !V->getType()->isPointerTy())
|
2010-02-05 22:03:18 +00:00
|
|
|
return CI;
|
|
|
|
|
|
|
|
// This is some kind of pointer constant. Turn it into a pointer-sized
|
|
|
|
// ConstantInt if possible.
|
|
|
|
const IntegerType *PtrTy = TD->getIntPtrType(V->getContext());
|
|
|
|
|
|
|
|
// Null pointer means 0, see SelectionDAGBuilder::getValue(const Value*).
|
|
|
|
if (isa<ConstantPointerNull>(V))
|
|
|
|
return ConstantInt::get(PtrTy, 0);
|
|
|
|
|
|
|
|
// IntToPtr const int.
|
|
|
|
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(V))
|
|
|
|
if (CE->getOpcode() == Instruction::IntToPtr)
|
|
|
|
if (ConstantInt *CI = dyn_cast<ConstantInt>(CE->getOperand(0))) {
|
|
|
|
// The constant is very likely to have the right type already.
|
|
|
|
if (CI->getType() == PtrTy)
|
|
|
|
return CI;
|
|
|
|
else
|
|
|
|
return cast<ConstantInt>
|
|
|
|
(ConstantExpr::getIntegerCast(CI, PtrTy, /*isSigned=*/false));
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2010-12-13 04:26:26 +00:00
|
|
|
/// GatherConstantCompares - Given a potentially 'or'd or 'and'd together
|
|
|
|
/// collection of icmp eq/ne instructions that compare a value against a
|
|
|
|
/// constant, return the value being compared, and stick the constant into the
|
|
|
|
/// Values vector.
|
2010-12-13 03:30:12 +00:00
|
|
|
static Value *
|
2010-12-13 04:26:26 +00:00
|
|
|
GatherConstantCompares(Value *V, std::vector<ConstantInt*> &Vals, Value *&Extra,
|
2011-02-07 22:37:28 +00:00
|
|
|
const TargetData *TD, bool isEQ, unsigned &UsedICmps) {
|
2010-12-13 04:26:26 +00:00
|
|
|
Instruction *I = dyn_cast<Instruction>(V);
|
|
|
|
if (I == 0) return 0;
|
2010-12-13 01:47:07 +00:00
|
|
|
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
// If this is an icmp against a constant, handle this as one of the cases.
|
2010-12-13 04:26:26 +00:00
|
|
|
if (ICmpInst *ICI = dyn_cast<ICmpInst>(I)) {
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
if (ConstantInt *C = GetConstantInt(I->getOperand(1), TD)) {
|
|
|
|
if (ICI->getPredicate() == (isEQ ? ICmpInst::ICMP_EQ:ICmpInst::ICMP_NE)) {
|
2011-02-07 22:37:28 +00:00
|
|
|
UsedICmps++;
|
2010-12-13 04:26:26 +00:00
|
|
|
Vals.push_back(C);
|
|
|
|
return I->getOperand(0);
|
|
|
|
}
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
|
|
|
|
// If we have "x ult 3" comparison, for example, then we can add 0,1,2 to
|
|
|
|
// the set.
|
|
|
|
ConstantRange Span =
|
2010-12-18 20:22:49 +00:00
|
|
|
ConstantRange::makeICmpRegion(ICI->getPredicate(), C->getValue());
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
|
|
|
|
// If this is an and/!= check then we want to optimize "x ugt 2" into
|
|
|
|
// x != 0 && x != 1.
|
|
|
|
if (!isEQ)
|
|
|
|
Span = Span.inverse();
|
|
|
|
|
|
|
|
// If there are a ton of values, we don't want to make a ginormous switch.
|
2010-12-17 10:48:14 +00:00
|
|
|
if (Span.getSetSize().ugt(8) || Span.isEmptySet() ||
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
// We don't handle wrapped sets yet.
|
|
|
|
Span.isWrappedSet())
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
for (APInt Tmp = Span.getLower(); Tmp != Span.getUpper(); ++Tmp)
|
|
|
|
Vals.push_back(ConstantInt::get(V->getContext(), Tmp));
|
2011-02-07 22:37:28 +00:00
|
|
|
UsedICmps++;
|
improve switch formation to handle small range
comparisons formed by comparisons. For example,
this:
void foo(unsigned x) {
if (x == 0 || x == 1 || x == 3 || x == 4 || x == 6)
bar();
}
compiles into:
_foo: ## @foo
## BB#0: ## %entry
cmpl $6, %edi
ja LBB0_2
## BB#1: ## %entry
movl %edi, %eax
movl $91, %ecx
btq %rax, %rcx
jb LBB0_3
instead of:
_foo: ## @foo
## BB#0: ## %entry
cmpl $2, %edi
jb LBB0_4
## BB#1: ## %switch.early.test
cmpl $6, %edi
ja LBB0_3
## BB#2: ## %switch.early.test
movl %edi, %eax
movl $88, %ecx
btq %rax, %rcx
jb LBB0_4
This catches a bunch of cases in GCC, which look like this:
%804 = load i32* @which_alternative, align 4, !tbaa !0
%805 = icmp ult i32 %804, 2
%806 = icmp eq i32 %804, 3
%or.cond121 = or i1 %805, %806
%807 = icmp eq i32 %804, 4
%or.cond124 = or i1 %or.cond121, %807
br i1 %or.cond124, label %.thread, label %808
turning this into a range comparison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@122045 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-17 06:20:15 +00:00
|
|
|
return I->getOperand(0);
|
|
|
|
}
|
2010-12-13 04:18:32 +00:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
// Otherwise, we can only handle an | or &, depending on isEQ.
|
2010-12-13 04:26:26 +00:00
|
|
|
if (I->getOpcode() != (isEQ ? Instruction::Or : Instruction::And))
|
2010-12-13 04:18:32 +00:00
|
|
|
return 0;
|
|
|
|
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
unsigned NumValsBeforeLHS = Vals.size();
|
2011-02-07 22:37:28 +00:00
|
|
|
unsigned UsedICmpsBeforeLHS = UsedICmps;
|
2010-12-13 04:26:26 +00:00
|
|
|
if (Value *LHS = GatherConstantCompares(I->getOperand(0), Vals, Extra, TD,
|
2011-02-07 22:37:28 +00:00
|
|
|
isEQ, UsedICmps)) {
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
unsigned NumVals = Vals.size();
|
2011-02-07 22:37:28 +00:00
|
|
|
unsigned UsedICmpsBeforeRHS = UsedICmps;
|
2010-12-13 04:26:26 +00:00
|
|
|
if (Value *RHS = GatherConstantCompares(I->getOperand(1), Vals, Extra, TD,
|
2011-02-07 22:37:28 +00:00
|
|
|
isEQ, UsedICmps)) {
|
2010-12-13 04:26:26 +00:00
|
|
|
if (LHS == RHS)
|
|
|
|
return LHS;
|
2010-12-13 07:41:29 +00:00
|
|
|
Vals.resize(NumVals);
|
2011-02-07 22:37:28 +00:00
|
|
|
UsedICmps = UsedICmpsBeforeRHS;
|
2010-12-13 04:26:26 +00:00
|
|
|
}
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
|
|
|
|
// The RHS of the or/and can't be folded in and we haven't used "Extra" yet,
|
|
|
|
// set it and return success.
|
|
|
|
if (Extra == 0 || Extra == I->getOperand(1)) {
|
|
|
|
Extra = I->getOperand(1);
|
|
|
|
return LHS;
|
|
|
|
}
|
|
|
|
|
|
|
|
Vals.resize(NumValsBeforeLHS);
|
2011-02-07 22:37:28 +00:00
|
|
|
UsedICmps = UsedICmpsBeforeLHS;
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If the LHS can't be folded in, but Extra is available and RHS can, try to
|
|
|
|
// use LHS as Extra.
|
|
|
|
if (Extra == 0 || Extra == I->getOperand(0)) {
|
2010-12-13 07:41:29 +00:00
|
|
|
Value *OldExtra = Extra;
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
Extra = I->getOperand(0);
|
|
|
|
if (Value *RHS = GatherConstantCompares(I->getOperand(1), Vals, Extra, TD,
|
2011-02-07 22:37:28 +00:00
|
|
|
isEQ, UsedICmps))
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
return RHS;
|
2010-12-13 07:41:29 +00:00
|
|
|
assert(Vals.size() == NumValsBeforeLHS);
|
|
|
|
Extra = OldExtra;
|
2008-02-20 11:26:25 +00:00
|
|
|
}
|
enhance the "change or icmp's into switch" xform to handle one value in an
'or sequence' that it doesn't understand. This allows us to optimize
something insane like this:
int crud (unsigned char c, unsigned x)
{
if(((((((((( (int) c <= 32 ||
(int) c == 46) || (int) c == 44)
|| (int) c == 58) || (int) c == 59) || (int) c == 60)
|| (int) c == 62) || (int) c == 34) || (int) c == 92)
|| (int) c == 39) != 0)
foo();
}
into:
define i32 @crud(i8 zeroext %c, i32 %x) nounwind ssp noredzone {
entry:
%cmp = icmp ult i8 %c, 33
br i1 %cmp, label %if.then, label %switch.early.test
switch.early.test: ; preds = %entry
switch i8 %c, label %if.end [
i8 39, label %if.then
i8 44, label %if.then
i8 58, label %if.then
i8 59, label %if.then
i8 60, label %if.then
i8 62, label %if.then
i8 46, label %if.then
i8 92, label %if.then
i8 34, label %if.then
]
by pulling the < comparison out ahead of the newly formed switch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121680 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 04:50:38 +00:00
|
|
|
|
2004-02-24 05:38:11 +00:00
|
|
|
return 0;
|
|
|
|
}
|
2010-12-13 04:26:26 +00:00
|
|
|
|
2008-12-16 20:54:32 +00:00
|
|
|
static void EraseTerminatorInstAndDCECond(TerminatorInst *TI) {
|
|
|
|
Instruction* Cond = 0;
|
|
|
|
if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
|
|
|
|
Cond = dyn_cast<Instruction>(SI->getCondition());
|
|
|
|
} else if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
|
|
|
|
if (BI->isConditional())
|
|
|
|
Cond = dyn_cast<Instruction>(BI->getCondition());
|
2010-12-05 18:29:03 +00:00
|
|
|
} else if (IndirectBrInst *IBI = dyn_cast<IndirectBrInst>(TI)) {
|
|
|
|
Cond = dyn_cast<Instruction>(IBI->getAddress());
|
2008-12-16 20:54:32 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
TI->eraseFromParent();
|
|
|
|
if (Cond) RecursivelyDeleteTriviallyDeadInstructions(Cond);
|
|
|
|
}
|
|
|
|
|
2008-11-27 23:25:44 +00:00
|
|
|
/// isValueEqualityComparison - Return true if the specified terminator checks
|
|
|
|
/// to see if a value is equal to constant integer value.
|
2010-02-05 22:03:18 +00:00
|
|
|
Value *SimplifyCFGOpt::isValueEqualityComparison(TerminatorInst *TI) {
|
|
|
|
Value *CV = 0;
|
2004-03-16 19:45:22 +00:00
|
|
|
if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
|
|
|
|
// Do not permit merging of large switch instructions into their
|
|
|
|
// predecessors unless there is only one predecessor.
|
2010-02-05 22:03:18 +00:00
|
|
|
if (SI->getNumSuccessors()*std::distance(pred_begin(SI->getParent()),
|
|
|
|
pred_end(SI->getParent())) <= 128)
|
|
|
|
CV = SI->getCondition();
|
|
|
|
} else if (BranchInst *BI = dyn_cast<BranchInst>(TI))
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
if (BI->isConditional() && BI->getCondition()->hasOneUse())
|
2006-12-23 06:05:41 +00:00
|
|
|
if (ICmpInst *ICI = dyn_cast<ICmpInst>(BI->getCondition()))
|
|
|
|
if ((ICI->getPredicate() == ICmpInst::ICMP_EQ ||
|
|
|
|
ICI->getPredicate() == ICmpInst::ICMP_NE) &&
|
2010-12-13 03:30:12 +00:00
|
|
|
GetConstantInt(ICI->getOperand(1), TD))
|
2010-02-05 22:03:18 +00:00
|
|
|
CV = ICI->getOperand(0);
|
|
|
|
|
|
|
|
// Unwrap any lossless ptrtoint cast.
|
|
|
|
if (TD && CV && CV->getType() == TD->getIntPtrType(CV->getContext()))
|
|
|
|
if (PtrToIntInst *PTII = dyn_cast<PtrToIntInst>(CV))
|
|
|
|
CV = PTII->getOperand(0);
|
|
|
|
return CV;
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
}
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// GetValueEqualityComparisonCases - Given a value comparison instruction,
|
|
|
|
/// decode all of the 'cases' that it represents and return the 'default' block.
|
2010-02-05 22:03:18 +00:00
|
|
|
BasicBlock *SimplifyCFGOpt::
|
2005-04-21 23:48:37 +00:00
|
|
|
GetValueEqualityComparisonCases(TerminatorInst *TI,
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
std::vector<std::pair<ConstantInt*,
|
|
|
|
BasicBlock*> > &Cases) {
|
|
|
|
if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
|
|
|
|
Cases.reserve(SI->getNumCases());
|
|
|
|
for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
|
2005-02-26 18:33:28 +00:00
|
|
|
Cases.push_back(std::make_pair(SI->getCaseValue(i), SI->getSuccessor(i)));
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
return SI->getDefaultDest();
|
|
|
|
}
|
|
|
|
|
|
|
|
BranchInst *BI = cast<BranchInst>(TI);
|
2006-12-23 06:05:41 +00:00
|
|
|
ICmpInst *ICI = cast<ICmpInst>(BI->getCondition());
|
2010-12-13 03:30:12 +00:00
|
|
|
Cases.push_back(std::make_pair(GetConstantInt(ICI->getOperand(1), TD),
|
2006-12-23 06:05:41 +00:00
|
|
|
BI->getSuccessor(ICI->getPredicate() ==
|
|
|
|
ICmpInst::ICMP_NE)));
|
|
|
|
return BI->getSuccessor(ICI->getPredicate() == ICmpInst::ICMP_EQ);
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// EliminateBlockCases - Given a vector of bb/value pairs, remove any entries
|
|
|
|
/// in the list that match the specified block.
|
2005-04-21 23:48:37 +00:00
|
|
|
static void EliminateBlockCases(BasicBlock *BB,
|
2005-02-24 06:17:52 +00:00
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > &Cases) {
|
|
|
|
for (unsigned i = 0, e = Cases.size(); i != e; ++i)
|
|
|
|
if (Cases[i].second == BB) {
|
|
|
|
Cases.erase(Cases.begin()+i);
|
|
|
|
--i; --e;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// ValuesOverlap - Return true if there are any keys in C1 that exist in C2 as
|
|
|
|
/// well.
|
2005-02-24 06:17:52 +00:00
|
|
|
static bool
|
|
|
|
ValuesOverlap(std::vector<std::pair<ConstantInt*, BasicBlock*> > &C1,
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > &C2) {
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > *V1 = &C1, *V2 = &C2;
|
|
|
|
|
|
|
|
// Make V1 be smaller than V2.
|
|
|
|
if (V1->size() > V2->size())
|
|
|
|
std::swap(V1, V2);
|
|
|
|
|
|
|
|
if (V1->size() == 0) return false;
|
|
|
|
if (V1->size() == 1) {
|
|
|
|
// Just scan V2.
|
|
|
|
ConstantInt *TheVal = (*V1)[0].first;
|
|
|
|
for (unsigned i = 0, e = V2->size(); i != e; ++i)
|
|
|
|
if (TheVal == (*V2)[i].first)
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Otherwise, just sort both lists and compare element by element.
|
2010-12-13 03:24:30 +00:00
|
|
|
array_pod_sort(V1->begin(), V1->end());
|
|
|
|
array_pod_sort(V2->begin(), V2->end());
|
2005-02-24 06:17:52 +00:00
|
|
|
unsigned i1 = 0, i2 = 0, e1 = V1->size(), e2 = V2->size();
|
|
|
|
while (i1 != e1 && i2 != e2) {
|
|
|
|
if ((*V1)[i1].first == (*V2)[i2].first)
|
|
|
|
return true;
|
|
|
|
if ((*V1)[i1].first < (*V2)[i2].first)
|
|
|
|
++i1;
|
|
|
|
else
|
|
|
|
++i2;
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// SimplifyEqualityComparisonWithOnlyPredecessor - If TI is known to be a
|
|
|
|
/// terminator instruction and its block is known to only have a single
|
|
|
|
/// predecessor block, check to see if that predecessor is also a value
|
|
|
|
/// comparison with the same value, and if that comparison determines the
|
|
|
|
/// outcome of this comparison. If so, simplify TI. This does a very limited
|
|
|
|
/// form of jump threading.
|
2010-02-05 22:03:18 +00:00
|
|
|
bool SimplifyCFGOpt::
|
|
|
|
SimplifyEqualityComparisonWithOnlyPredecessor(TerminatorInst *TI,
|
|
|
|
BasicBlock *Pred) {
|
2005-02-24 06:17:52 +00:00
|
|
|
Value *PredVal = isValueEqualityComparison(Pred->getTerminator());
|
|
|
|
if (!PredVal) return false; // Not a value comparison in predecessor.
|
|
|
|
|
|
|
|
Value *ThisVal = isValueEqualityComparison(TI);
|
|
|
|
assert(ThisVal && "This isn't a value comparison!!");
|
|
|
|
if (ThisVal != PredVal) return false; // Different predicates.
|
|
|
|
|
|
|
|
// Find out information about when control will move from Pred to TI's block.
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
|
|
|
|
BasicBlock *PredDef = GetValueEqualityComparisonCases(Pred->getTerminator(),
|
|
|
|
PredCases);
|
|
|
|
EliminateBlockCases(PredDef, PredCases); // Remove default from cases.
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2005-02-24 06:17:52 +00:00
|
|
|
// Find information about how control leaves this block.
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > ThisCases;
|
|
|
|
BasicBlock *ThisDef = GetValueEqualityComparisonCases(TI, ThisCases);
|
|
|
|
EliminateBlockCases(ThisDef, ThisCases); // Remove default from cases.
|
|
|
|
|
|
|
|
// If TI's block is the default block from Pred's comparison, potentially
|
|
|
|
// simplify TI based on this knowledge.
|
|
|
|
if (PredDef == TI->getParent()) {
|
|
|
|
// If we are here, we know that the value is none of those cases listed in
|
|
|
|
// PredCases. If there are any cases in ThisCases that are in PredCases, we
|
|
|
|
// can simplify TI.
|
2010-12-13 01:47:07 +00:00
|
|
|
if (!ValuesOverlap(PredCases, ThisCases))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
if (isa<BranchInst>(TI)) {
|
|
|
|
// Okay, one of the successors of this condbr is dead. Convert it to a
|
|
|
|
// uncond br.
|
|
|
|
assert(ThisCases.size() == 1 && "Branch can only have one case!");
|
|
|
|
// Insert the new branch.
|
|
|
|
Instruction *NI = BranchInst::Create(ThisDef, TI);
|
|
|
|
(void) NI;
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Remove PHI node entries for the dead edge.
|
|
|
|
ThisCases[0].second->removePredecessor(TI->getParent());
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
|
|
|
|
<< "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
EraseTerminatorInstAndDCECond(TI);
|
|
|
|
return true;
|
2005-02-24 06:17:52 +00:00
|
|
|
}
|
2010-12-13 01:47:07 +00:00
|
|
|
|
|
|
|
SwitchInst *SI = cast<SwitchInst>(TI);
|
|
|
|
// Okay, TI has cases that are statically dead, prune them away.
|
|
|
|
SmallPtrSet<Constant*, 16> DeadCases;
|
2005-02-24 06:17:52 +00:00
|
|
|
for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
|
2010-12-13 01:47:07 +00:00
|
|
|
DeadCases.insert(PredCases[i].first);
|
|
|
|
|
|
|
|
DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
|
|
|
|
<< "Through successor TI: " << *TI);
|
|
|
|
|
|
|
|
for (unsigned i = SI->getNumCases()-1; i != 0; --i)
|
|
|
|
if (DeadCases.count(SI->getCaseValue(i))) {
|
|
|
|
SI->getSuccessor(i)->removePredecessor(TI->getParent());
|
|
|
|
SI->removeCase(i);
|
2005-02-24 06:17:52 +00:00
|
|
|
}
|
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
DEBUG(dbgs() << "Leaving: " << *TI << "\n");
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Otherwise, TI's block must correspond to some matched value. Find out
|
|
|
|
// which value (or set of values) this is.
|
|
|
|
ConstantInt *TIV = 0;
|
|
|
|
BasicBlock *TIBB = TI->getParent();
|
|
|
|
for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
|
|
|
|
if (PredCases[i].second == TIBB) {
|
|
|
|
if (TIV != 0)
|
|
|
|
return false; // Cannot handle multiple values coming to this block.
|
|
|
|
TIV = PredCases[i].first;
|
|
|
|
}
|
|
|
|
assert(TIV && "No edge from pred to succ?");
|
|
|
|
|
|
|
|
// Okay, we found the one constant that our value can be if we get into TI's
|
|
|
|
// BB. Find out which successor will unconditionally be branched to.
|
|
|
|
BasicBlock *TheRealDest = 0;
|
|
|
|
for (unsigned i = 0, e = ThisCases.size(); i != e; ++i)
|
|
|
|
if (ThisCases[i].first == TIV) {
|
|
|
|
TheRealDest = ThisCases[i].second;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If not handled by any explicit cases, it is handled by the default case.
|
|
|
|
if (TheRealDest == 0) TheRealDest = ThisDef;
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Remove PHI node entries for dead edges.
|
|
|
|
BasicBlock *CheckEdge = TheRealDest;
|
|
|
|
for (succ_iterator SI = succ_begin(TIBB), e = succ_end(TIBB); SI != e; ++SI)
|
|
|
|
if (*SI != CheckEdge)
|
|
|
|
(*SI)->removePredecessor(TIBB);
|
|
|
|
else
|
|
|
|
CheckEdge = 0;
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Insert the new branch.
|
|
|
|
Instruction *NI = BranchInst::Create(TheRealDest, TI);
|
|
|
|
(void) NI;
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
|
|
|
|
<< "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
|
2005-02-24 06:17:52 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
EraseTerminatorInstAndDCECond(TI);
|
|
|
|
return true;
|
2005-02-24 06:17:52 +00:00
|
|
|
}
|
|
|
|
|
2009-03-12 21:01:11 +00:00
|
|
|
namespace {
|
|
|
|
/// ConstantIntOrdering - This class implements a stable ordering of constant
|
|
|
|
/// integers that does not depend on their address. This is important for
|
|
|
|
/// applications that sort ConstantInt's to ensure uniqueness.
|
|
|
|
struct ConstantIntOrdering {
|
|
|
|
bool operator()(const ConstantInt *LHS, const ConstantInt *RHS) const {
|
|
|
|
return LHS->getValue().ult(RHS->getValue());
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
2009-03-12 01:00:26 +00:00
|
|
|
|
2010-12-13 02:00:58 +00:00
|
|
|
static int ConstantIntSortPredicate(const void *P1, const void *P2) {
|
|
|
|
const ConstantInt *LHS = *(const ConstantInt**)P1;
|
|
|
|
const ConstantInt *RHS = *(const ConstantInt**)P2;
|
2010-12-15 04:52:41 +00:00
|
|
|
if (LHS->getValue().ult(RHS->getValue()))
|
|
|
|
return 1;
|
|
|
|
if (LHS->getValue() == RHS->getValue())
|
|
|
|
return 0;
|
|
|
|
return -1;
|
2010-12-13 02:00:58 +00:00
|
|
|
}
|
|
|
|
|
2009-01-19 23:43:56 +00:00
|
|
|
/// FoldValueComparisonIntoPredecessors - The specified terminator is a value
|
|
|
|
/// equality comparison instruction (either a switch or a branch on "X == c").
|
|
|
|
/// See if any of the predecessors of the terminator block are value comparisons
|
|
|
|
/// on the same value. If so, and if safe to do so, fold them together.
|
2010-02-05 22:03:18 +00:00
|
|
|
bool SimplifyCFGOpt::FoldValueComparisonIntoPredecessors(TerminatorInst *TI) {
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
BasicBlock *BB = TI->getParent();
|
|
|
|
Value *CV = isValueEqualityComparison(TI); // CondVal
|
|
|
|
assert(CV && "Not a comparison?");
|
|
|
|
bool Changed = false;
|
|
|
|
|
2008-02-18 07:42:56 +00:00
|
|
|
SmallVector<BasicBlock*, 16> Preds(pred_begin(BB), pred_end(BB));
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
while (!Preds.empty()) {
|
2009-05-06 17:22:41 +00:00
|
|
|
BasicBlock *Pred = Preds.pop_back_val();
|
2005-04-21 23:48:37 +00:00
|
|
|
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
// See if the predecessor is a comparison with the same value.
|
|
|
|
TerminatorInst *PTI = Pred->getTerminator();
|
|
|
|
Value *PCV = isValueEqualityComparison(PTI); // PredCondVal
|
|
|
|
|
|
|
|
if (PCV == CV && SafeToMergeTerminators(TI, PTI)) {
|
|
|
|
// Figure out which 'cases' to copy from SI to PSI.
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > BBCases;
|
|
|
|
BasicBlock *BBDefault = GetValueEqualityComparisonCases(TI, BBCases);
|
|
|
|
|
|
|
|
std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
|
|
|
|
BasicBlock *PredDefault = GetValueEqualityComparisonCases(PTI, PredCases);
|
|
|
|
|
|
|
|
// Based on whether the default edge from PTI goes to BB or not, fill in
|
|
|
|
// PredCases and PredDefault with the new switch cases we would like to
|
|
|
|
// build.
|
2008-02-18 07:42:56 +00:00
|
|
|
SmallVector<BasicBlock*, 8> NewSuccessors;
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
|
|
|
|
if (PredDefault == BB) {
|
|
|
|
// If this is the default destination from PTI, only the edges in TI
|
|
|
|
// that don't occur in PTI, or that branch to BB will be activated.
|
2009-03-12 21:01:11 +00:00
|
|
|
std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
|
|
|
|
if (PredCases[i].second != BB)
|
|
|
|
PTIHandled.insert(PredCases[i].first);
|
|
|
|
else {
|
|
|
|
// The default destination is BB, we don't need explicit targets.
|
|
|
|
std::swap(PredCases[i], PredCases.back());
|
|
|
|
PredCases.pop_back();
|
|
|
|
--i; --e;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Reconstruct the new switch statement we will be building.
|
|
|
|
if (PredDefault != BBDefault) {
|
|
|
|
PredDefault->removePredecessor(Pred);
|
|
|
|
PredDefault = BBDefault;
|
|
|
|
NewSuccessors.push_back(BBDefault);
|
|
|
|
}
|
|
|
|
for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
|
|
|
|
if (!PTIHandled.count(BBCases[i].first) &&
|
|
|
|
BBCases[i].second != BBDefault) {
|
|
|
|
PredCases.push_back(BBCases[i]);
|
|
|
|
NewSuccessors.push_back(BBCases[i].second);
|
|
|
|
}
|
|
|
|
|
|
|
|
} else {
|
|
|
|
// If this is not the default destination from PSI, only the edges
|
|
|
|
// in SI that occur in PSI with a destination of BB will be
|
|
|
|
// activated.
|
2009-03-12 21:01:11 +00:00
|
|
|
std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
|
|
|
|
if (PredCases[i].second == BB) {
|
|
|
|
PTIHandled.insert(PredCases[i].first);
|
|
|
|
std::swap(PredCases[i], PredCases.back());
|
|
|
|
PredCases.pop_back();
|
|
|
|
--i; --e;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Okay, now we know which constants were sent to BB from the
|
|
|
|
// predecessor. Figure out where they will all go now.
|
|
|
|
for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
|
|
|
|
if (PTIHandled.count(BBCases[i].first)) {
|
|
|
|
// If this is one we are capable of getting...
|
|
|
|
PredCases.push_back(BBCases[i]);
|
|
|
|
NewSuccessors.push_back(BBCases[i].second);
|
|
|
|
PTIHandled.erase(BBCases[i].first);// This constant is taken care of
|
|
|
|
}
|
|
|
|
|
|
|
|
// If there are any constants vectored to BB that TI doesn't handle,
|
|
|
|
// they must go to the default destination of TI.
|
2009-03-12 21:01:11 +00:00
|
|
|
for (std::set<ConstantInt*, ConstantIntOrdering>::iterator I =
|
|
|
|
PTIHandled.begin(),
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
E = PTIHandled.end(); I != E; ++I) {
|
|
|
|
PredCases.push_back(std::make_pair(*I, BBDefault));
|
|
|
|
NewSuccessors.push_back(BBDefault);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Okay, at this point, we know which new successor Pred will get. Make
|
|
|
|
// sure we update the number of entries in the PHI nodes for these
|
|
|
|
// successors.
|
|
|
|
for (unsigned i = 0, e = NewSuccessors.size(); i != e; ++i)
|
|
|
|
AddPredecessorToBlock(NewSuccessors[i], Pred, BB);
|
|
|
|
|
2010-02-05 22:03:18 +00:00
|
|
|
// Convert pointer to int before we switch.
|
2010-02-16 11:11:14 +00:00
|
|
|
if (CV->getType()->isPointerTy()) {
|
2010-02-05 22:03:18 +00:00
|
|
|
assert(TD && "Cannot switch on pointer without TargetData");
|
|
|
|
CV = new PtrToIntInst(CV, TD->getIntPtrType(CV->getContext()),
|
|
|
|
"magicptr", PTI);
|
|
|
|
}
|
|
|
|
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
// Now that the successors are updated, create the new Switch instruction.
|
2008-05-15 10:04:30 +00:00
|
|
|
SwitchInst *NewSI = SwitchInst::Create(CV, PredDefault,
|
|
|
|
PredCases.size(), PTI);
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
|
|
|
|
NewSI->addCase(PredCases[i].first, PredCases[i].second);
|
2005-01-01 16:02:12 +00:00
|
|
|
|
2008-12-16 20:54:32 +00:00
|
|
|
EraseTerminatorInstAndDCECond(PTI);
|
2005-01-01 16:02:12 +00:00
|
|
|
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
// Okay, last check. If BB is still a successor of PSI, then we must
|
|
|
|
// have an infinite loop case. If so, add an infinitely looping block
|
|
|
|
// to handle the case to preserve the behavior of the code.
|
|
|
|
BasicBlock *InfLoopBlock = 0;
|
|
|
|
for (unsigned i = 0, e = NewSI->getNumSuccessors(); i != e; ++i)
|
|
|
|
if (NewSI->getSuccessor(i) == BB) {
|
|
|
|
if (InfLoopBlock == 0) {
|
2008-07-13 22:23:11 +00:00
|
|
|
// Insert it at the end of the function, because it's either code,
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
// or it won't matter if it's hot. :)
|
2009-08-13 21:58:54 +00:00
|
|
|
InfLoopBlock = BasicBlock::Create(BB->getContext(),
|
|
|
|
"infloop", BB->getParent());
|
2008-04-06 20:25:17 +00:00
|
|
|
BranchInst::Create(InfLoopBlock, InfLoopBlock);
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
}
|
|
|
|
NewSI->setSuccessor(i, InfLoopBlock);
|
|
|
|
}
|
2005-04-21 23:48:37 +00:00
|
|
|
|
Implement switch->br and br->switch folding by ripping out the switch->switch
and br->br code and generalizing it. This allows us to compile code like this:
int test(Instruction *I) {
if (isa<CastInst>(I))
return foo(7);
else if (isa<BranchInst>(I))
return foo(123);
else if (isa<UnwindInst>(I))
return foo(1241);
else if (isa<SetCondInst>(I))
return foo(1);
else if (isa<VAArgInst>(I))
return foo(42);
return foo(-1);
}
into:
int %_Z4testPN4llvm11InstructionE("struct.llvm::Instruction"* %I) {
entry:
%tmp.1.i.i.i.i.i.i.i = getelementptr "struct.llvm::Instruction"* %I, long 0, ubyte 4 ; <uint*> [#uses=1]
%tmp.2.i.i.i.i.i.i.i = load uint* %tmp.1.i.i.i.i.i.i.i ; <uint> [#uses=2]
%tmp.2.i.i.i.i.i.i = seteq uint %tmp.2.i.i.i.i.i.i.i, 27 ; <bool> [#uses=0]
switch uint %tmp.2.i.i.i.i.i.i.i, label %endif.0 [
uint 27, label %then.0
uint 2, label %then.1
uint 5, label %then.2
uint 14, label %then.3
uint 15, label %then.3
uint 16, label %then.3
uint 17, label %then.3
uint 18, label %then.3
uint 19, label %then.3
uint 32, label %then.4
]
...
As well as handling the cases in 176.gcc and many other programs more effectively.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11964 91177308-0d34-0410-b5e6-96231b3b80d8
2004-02-28 21:28:10 +00:00
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return Changed;
|
|
|
|
}
|
|
|
|
|
2009-06-15 20:59:27 +00:00
|
|
|
// isSafeToHoistInvoke - If we would need to insert a select that uses the
|
|
|
|
// value of this invoke (comments in HoistThenElseCodeToIf explain why we
|
|
|
|
// would need to do this), we can't hoist the invoke, as there is nowhere
|
|
|
|
// to put the select in this case.
|
|
|
|
static bool isSafeToHoistInvoke(BasicBlock *BB1, BasicBlock *BB2,
|
|
|
|
Instruction *I1, Instruction *I2) {
|
|
|
|
for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
|
|
|
|
PHINode *PN;
|
|
|
|
for (BasicBlock::iterator BBI = SI->begin();
|
|
|
|
(PN = dyn_cast<PHINode>(BBI)); ++BBI) {
|
|
|
|
Value *BB1V = PN->getIncomingValueForBlock(BB1);
|
|
|
|
Value *BB2V = PN->getIncomingValueForBlock(BB2);
|
|
|
|
if (BB1V != BB2V && (BB1V==I1 || BB2V==I2)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2005-08-03 17:59:45 +00:00
|
|
|
/// HoistThenElseCodeToIf - Given a conditional branch that goes to BB1 and
|
2004-11-30 00:29:14 +00:00
|
|
|
/// BB2, hoist any common code in the two blocks up into the branch block. The
|
|
|
|
/// caller of this function guarantees that BI's block dominates BB1 and BB2.
|
|
|
|
static bool HoistThenElseCodeToIf(BranchInst *BI) {
|
|
|
|
// This does very trivial matching, with limited scanning, to find identical
|
|
|
|
// instructions in the two blocks. In particular, we don't want to get into
|
|
|
|
// O(M*N) situations here where M and N are the sizes of BB1 and BB2. As
|
|
|
|
// such, we currently just scan for obviously identical instructions in an
|
|
|
|
// identical order.
|
|
|
|
BasicBlock *BB1 = BI->getSuccessor(0); // The true destination.
|
|
|
|
BasicBlock *BB2 = BI->getSuccessor(1); // The false destination
|
|
|
|
|
2009-02-04 00:03:08 +00:00
|
|
|
BasicBlock::iterator BB1_Itr = BB1->begin();
|
|
|
|
BasicBlock::iterator BB2_Itr = BB2->begin();
|
|
|
|
|
|
|
|
Instruction *I1 = BB1_Itr++, *I2 = BB2_Itr++;
|
|
|
|
while (isa<DbgInfoIntrinsic>(I1))
|
|
|
|
I1 = BB1_Itr++;
|
|
|
|
while (isa<DbgInfoIntrinsic>(I2))
|
|
|
|
I2 = BB2_Itr++;
|
2009-06-15 20:59:27 +00:00
|
|
|
if (I1->getOpcode() != I2->getOpcode() || isa<PHINode>(I1) ||
|
2009-08-25 22:11:20 +00:00
|
|
|
!I1->isIdenticalToWhenDefined(I2) ||
|
2009-06-15 20:59:27 +00:00
|
|
|
(isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2)))
|
2004-11-30 00:29:14 +00:00
|
|
|
return false;
|
|
|
|
|
|
|
|
// If we get here, we can hoist at least one instruction.
|
|
|
|
BasicBlock *BIParent = BI->getParent();
|
|
|
|
|
|
|
|
do {
|
|
|
|
// If we are hoisting the terminator instruction, don't move one (making a
|
|
|
|
// broken BB), instead clone it, and remove BI.
|
|
|
|
if (isa<TerminatorInst>(I1))
|
|
|
|
goto HoistTerminator;
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2004-11-30 00:29:14 +00:00
|
|
|
// For a normal instruction, we just move one to right before the branch,
|
|
|
|
// then replace all uses of the other with the first. Finally, we remove
|
|
|
|
// the now redundant second instruction.
|
|
|
|
BIParent->getInstList().splice(BI, BB1->getInstList(), I1);
|
|
|
|
if (!I2->use_empty())
|
|
|
|
I2->replaceAllUsesWith(I1);
|
2009-08-25 22:11:20 +00:00
|
|
|
I1->intersectOptionalDataWith(I2);
|
2010-12-14 06:17:25 +00:00
|
|
|
I2->eraseFromParent();
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2009-02-04 00:03:08 +00:00
|
|
|
I1 = BB1_Itr++;
|
|
|
|
while (isa<DbgInfoIntrinsic>(I1))
|
|
|
|
I1 = BB1_Itr++;
|
|
|
|
I2 = BB2_Itr++;
|
|
|
|
while (isa<DbgInfoIntrinsic>(I2))
|
|
|
|
I2 = BB2_Itr++;
|
2009-08-25 22:11:20 +00:00
|
|
|
} while (I1->getOpcode() == I2->getOpcode() &&
|
|
|
|
I1->isIdenticalToWhenDefined(I2));
|
2004-11-30 00:29:14 +00:00
|
|
|
|
|
|
|
return true;
|
|
|
|
|
|
|
|
HoistTerminator:
|
2009-06-15 20:59:27 +00:00
|
|
|
// It may not be possible to hoist an invoke.
|
|
|
|
if (isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2))
|
|
|
|
return true;
|
|
|
|
|
2004-11-30 00:29:14 +00:00
|
|
|
// Okay, it is safe to hoist the terminator.
|
2009-09-27 07:38:41 +00:00
|
|
|
Instruction *NT = I1->clone();
|
2004-11-30 00:29:14 +00:00
|
|
|
BIParent->getInstList().insert(BI, NT);
|
2010-01-05 13:12:22 +00:00
|
|
|
if (!NT->getType()->isVoidTy()) {
|
2004-11-30 00:29:14 +00:00
|
|
|
I1->replaceAllUsesWith(NT);
|
|
|
|
I2->replaceAllUsesWith(NT);
|
2007-02-11 01:37:51 +00:00
|
|
|
NT->takeName(I1);
|
2004-11-30 00:29:14 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// Hoisting one of the terminators from our successor is a great thing.
|
|
|
|
// Unfortunately, the successors of the if/else blocks may have PHI nodes in
|
|
|
|
// them. If they do, all PHI entries for BB1/BB2 must agree for all PHI
|
|
|
|
// nodes, so we insert select instruction to compute the final result.
|
|
|
|
std::map<std::pair<Value*,Value*>, SelectInst*> InsertedSelects;
|
|
|
|
for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
|
|
|
|
PHINode *PN;
|
|
|
|
for (BasicBlock::iterator BBI = SI->begin();
|
2004-11-30 07:47:34 +00:00
|
|
|
(PN = dyn_cast<PHINode>(BBI)); ++BBI) {
|
2004-11-30 00:29:14 +00:00
|
|
|
Value *BB1V = PN->getIncomingValueForBlock(BB1);
|
|
|
|
Value *BB2V = PN->getIncomingValueForBlock(BB2);
|
2010-12-13 01:47:07 +00:00
|
|
|
if (BB1V == BB2V) continue;
|
|
|
|
|
|
|
|
// These values do not agree. Insert a select instruction before NT
|
|
|
|
// that determines the right value.
|
|
|
|
SelectInst *&SI = InsertedSelects[std::make_pair(BB1V, BB2V)];
|
|
|
|
if (SI == 0)
|
|
|
|
SI = SelectInst::Create(BI->getCondition(), BB1V, BB2V,
|
|
|
|
BB1V->getName()+"."+BB2V->getName(), NT);
|
|
|
|
// Make the PHI node use the select for all incoming values for BB1/BB2
|
|
|
|
for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
|
|
|
|
if (PN->getIncomingBlock(i) == BB1 || PN->getIncomingBlock(i) == BB2)
|
|
|
|
PN->setIncomingValue(i, SI);
|
2004-11-30 00:29:14 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Update any PHI nodes in our new successors.
|
|
|
|
for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI)
|
|
|
|
AddPredecessorToBlock(*SI, BIParent, BB1);
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2008-12-16 20:54:32 +00:00
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
2004-11-30 00:29:14 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2008-06-07 08:52:29 +00:00
|
|
|
/// SpeculativelyExecuteBB - Given a conditional branch that goes to BB1
|
|
|
|
/// and an BB2 and the only successor of BB1 is BB2, hoist simple code
|
|
|
|
/// (for now, restricted to a single instruction that's side effect free) from
|
|
|
|
/// the BB1 into the branch block to speculatively execute it.
|
|
|
|
static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *BB1) {
|
|
|
|
// Only speculatively execution a single instruction (not counting the
|
|
|
|
// terminator) for now.
|
2009-03-06 06:00:17 +00:00
|
|
|
Instruction *HInst = NULL;
|
|
|
|
Instruction *Term = BB1->getTerminator();
|
|
|
|
for (BasicBlock::iterator BBI = BB1->begin(), BBE = BB1->end();
|
|
|
|
BBI != BBE; ++BBI) {
|
|
|
|
Instruction *I = BBI;
|
|
|
|
// Skip debug info.
|
2010-12-13 01:47:07 +00:00
|
|
|
if (isa<DbgInfoIntrinsic>(I)) continue;
|
|
|
|
if (I == Term) break;
|
2009-03-06 06:00:17 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
if (HInst)
|
2009-03-06 06:00:17 +00:00
|
|
|
return false;
|
2010-12-13 01:47:07 +00:00
|
|
|
HInst = I;
|
2009-03-06 06:00:17 +00:00
|
|
|
}
|
|
|
|
if (!HInst)
|
|
|
|
return false;
|
2008-06-07 08:52:29 +00:00
|
|
|
|
2008-06-11 19:18:20 +00:00
|
|
|
// Be conservative for now. FP select instruction can often be expensive.
|
|
|
|
Value *BrCond = BI->getCondition();
|
2010-12-13 01:47:07 +00:00
|
|
|
if (isa<FCmpInst>(BrCond))
|
2008-06-11 19:18:20 +00:00
|
|
|
return false;
|
|
|
|
|
2008-06-07 08:52:29 +00:00
|
|
|
// If BB1 is actually on the false edge of the conditional branch, remember
|
|
|
|
// to swap the select operands later.
|
|
|
|
bool Invert = false;
|
|
|
|
if (BB1 != BI->getSuccessor(0)) {
|
|
|
|
assert(BB1 == BI->getSuccessor(1) && "No edge from 'if' block?");
|
|
|
|
Invert = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Turn
|
|
|
|
// BB:
|
|
|
|
// %t1 = icmp
|
|
|
|
// br i1 %t1, label %BB1, label %BB2
|
|
|
|
// BB1:
|
|
|
|
// %t3 = add %t2, c
|
|
|
|
// br label BB2
|
|
|
|
// BB2:
|
|
|
|
// =>
|
|
|
|
// BB:
|
|
|
|
// %t1 = icmp
|
|
|
|
// %t4 = add %t2, c
|
|
|
|
// %t3 = select i1 %t1, %t2, %t3
|
2009-03-06 06:00:17 +00:00
|
|
|
switch (HInst->getOpcode()) {
|
2008-06-07 08:52:29 +00:00
|
|
|
default: return false; // Not safe / profitable to hoist.
|
|
|
|
case Instruction::Add:
|
|
|
|
case Instruction::Sub:
|
2009-06-04 22:49:04 +00:00
|
|
|
// Not worth doing for vector ops.
|
2010-02-16 11:11:14 +00:00
|
|
|
if (HInst->getType()->isVectorTy())
|
2009-01-18 23:22:07 +00:00
|
|
|
return false;
|
|
|
|
break;
|
2008-06-07 08:52:29 +00:00
|
|
|
case Instruction::And:
|
|
|
|
case Instruction::Or:
|
|
|
|
case Instruction::Xor:
|
|
|
|
case Instruction::Shl:
|
|
|
|
case Instruction::LShr:
|
|
|
|
case Instruction::AShr:
|
2009-01-18 23:22:07 +00:00
|
|
|
// Don't mess with vector operations.
|
2010-02-16 11:11:14 +00:00
|
|
|
if (HInst->getType()->isVectorTy())
|
2008-06-25 07:50:12 +00:00
|
|
|
return false;
|
2008-06-07 08:52:29 +00:00
|
|
|
break; // These are all cheap and non-trapping instructions.
|
|
|
|
}
|
2009-01-19 00:36:37 +00:00
|
|
|
|
|
|
|
// If the instruction is obviously dead, don't try to predicate it.
|
2009-03-06 06:00:17 +00:00
|
|
|
if (HInst->use_empty()) {
|
|
|
|
HInst->eraseFromParent();
|
2009-01-19 00:36:37 +00:00
|
|
|
return true;
|
|
|
|
}
|
2008-06-07 08:52:29 +00:00
|
|
|
|
|
|
|
// Can we speculatively execute the instruction? And what is the value
|
|
|
|
// if the condition is false? Consider the phi uses, if the incoming value
|
|
|
|
// from the "if" block are all the same V, then V is the value of the
|
|
|
|
// select if the condition is false.
|
|
|
|
BasicBlock *BIParent = BI->getParent();
|
|
|
|
SmallVector<PHINode*, 4> PHIUses;
|
|
|
|
Value *FalseV = NULL;
|
2009-01-19 00:36:37 +00:00
|
|
|
|
|
|
|
BasicBlock *BB2 = BB1->getTerminator()->getSuccessor(0);
|
2009-03-06 06:00:17 +00:00
|
|
|
for (Value::use_iterator UI = HInst->use_begin(), E = HInst->use_end();
|
2008-06-07 08:52:29 +00:00
|
|
|
UI != E; ++UI) {
|
2009-01-19 00:36:37 +00:00
|
|
|
// Ignore any user that is not a PHI node in BB2. These can only occur in
|
|
|
|
// unreachable blocks, because they would not be dominated by the instr.
|
2010-07-22 11:43:44 +00:00
|
|
|
PHINode *PN = dyn_cast<PHINode>(*UI);
|
2009-01-19 00:36:37 +00:00
|
|
|
if (!PN || PN->getParent() != BB2)
|
|
|
|
return false;
|
2008-06-07 08:52:29 +00:00
|
|
|
PHIUses.push_back(PN);
|
2009-01-19 00:36:37 +00:00
|
|
|
|
2008-06-07 08:52:29 +00:00
|
|
|
Value *PHIV = PN->getIncomingValueForBlock(BIParent);
|
|
|
|
if (!FalseV)
|
|
|
|
FalseV = PHIV;
|
|
|
|
else if (FalseV != PHIV)
|
2009-01-19 00:36:37 +00:00
|
|
|
return false; // Inconsistent value when condition is false.
|
2008-06-07 08:52:29 +00:00
|
|
|
}
|
2009-01-19 00:36:37 +00:00
|
|
|
|
|
|
|
assert(FalseV && "Must have at least one user, and it must be a PHI");
|
2008-06-07 08:52:29 +00:00
|
|
|
|
2008-06-12 21:15:59 +00:00
|
|
|
// Do not hoist the instruction if any of its operands are defined but not
|
|
|
|
// used in this BB. The transformation will prevent the operand from
|
|
|
|
// being sunk into the use block.
|
2009-03-06 06:00:17 +00:00
|
|
|
for (User::op_iterator i = HInst->op_begin(), e = HInst->op_end();
|
|
|
|
i != e; ++i) {
|
2008-06-12 21:15:59 +00:00
|
|
|
Instruction *OpI = dyn_cast<Instruction>(*i);
|
|
|
|
if (OpI && OpI->getParent() == BIParent &&
|
|
|
|
!OpI->isUsedInBasicBlock(BIParent))
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2008-09-18 22:50:42 +00:00
|
|
|
// If we get here, we can hoist the instruction. Try to place it
|
2009-03-13 01:05:24 +00:00
|
|
|
// before the icmp instruction preceding the conditional branch.
|
2008-09-18 22:50:42 +00:00
|
|
|
BasicBlock::iterator InsertPos = BI;
|
2009-03-13 01:05:24 +00:00
|
|
|
if (InsertPos != BIParent->begin())
|
|
|
|
--InsertPos;
|
|
|
|
// Skip debug info between condition and branch.
|
|
|
|
while (InsertPos != BIParent->begin() && isa<DbgInfoIntrinsic>(InsertPos))
|
2008-09-18 22:50:42 +00:00
|
|
|
--InsertPos;
|
2008-10-03 18:57:37 +00:00
|
|
|
if (InsertPos == BrCond && !isa<PHINode>(BrCond)) {
|
2008-09-18 22:50:42 +00:00
|
|
|
SmallPtrSet<Instruction *, 4> BB1Insns;
|
|
|
|
for(BasicBlock::iterator BB1I = BB1->begin(), BB1E = BB1->end();
|
|
|
|
BB1I != BB1E; ++BB1I)
|
|
|
|
BB1Insns.insert(BB1I);
|
|
|
|
for(Value::use_iterator UI = BrCond->use_begin(), UE = BrCond->use_end();
|
|
|
|
UI != UE; ++UI) {
|
|
|
|
Instruction *Use = cast<Instruction>(*UI);
|
2010-12-13 01:47:07 +00:00
|
|
|
if (!BB1Insns.count(Use)) continue;
|
|
|
|
|
|
|
|
// If BrCond uses the instruction that place it just before
|
|
|
|
// branch instruction.
|
|
|
|
InsertPos = BI;
|
|
|
|
break;
|
2008-09-18 22:50:42 +00:00
|
|
|
}
|
|
|
|
} else
|
|
|
|
InsertPos = BI;
|
2009-03-06 06:00:17 +00:00
|
|
|
BIParent->getInstList().splice(InsertPos, BB1->getInstList(), HInst);
|
2008-06-07 08:52:29 +00:00
|
|
|
|
|
|
|
// Create a select whose true value is the speculatively executed value and
|
|
|
|
// false value is the previously determined FalseV.
|
|
|
|
SelectInst *SI;
|
|
|
|
if (Invert)
|
2009-03-06 06:00:17 +00:00
|
|
|
SI = SelectInst::Create(BrCond, FalseV, HInst,
|
|
|
|
FalseV->getName() + "." + HInst->getName(), BI);
|
2008-06-07 08:52:29 +00:00
|
|
|
else
|
2009-03-06 06:00:17 +00:00
|
|
|
SI = SelectInst::Create(BrCond, HInst, FalseV,
|
|
|
|
HInst->getName() + "." + FalseV->getName(), BI);
|
2008-06-07 08:52:29 +00:00
|
|
|
|
|
|
|
// Make the PHI node use the select for all incoming values for "then" and
|
|
|
|
// "if" blocks.
|
|
|
|
for (unsigned i = 0, e = PHIUses.size(); i != e; ++i) {
|
|
|
|
PHINode *PN = PHIUses[i];
|
|
|
|
for (unsigned j = 0, ee = PN->getNumIncomingValues(); j != ee; ++j)
|
2010-12-13 01:47:07 +00:00
|
|
|
if (PN->getIncomingBlock(j) == BB1 || PN->getIncomingBlock(j) == BIParent)
|
2008-06-07 08:52:29 +00:00
|
|
|
PN->setIncomingValue(j, SI);
|
|
|
|
}
|
|
|
|
|
2008-06-12 21:15:59 +00:00
|
|
|
++NumSpeculations;
|
2008-06-07 08:52:29 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2005-09-20 00:43:16 +00:00
|
|
|
/// BlockIsSimpleEnoughToThreadThrough - Return true if we can thread a branch
|
|
|
|
/// across this block.
|
|
|
|
static bool BlockIsSimpleEnoughToThreadThrough(BasicBlock *BB) {
|
|
|
|
BranchInst *BI = cast<BranchInst>(BB->getTerminator());
|
2005-09-20 01:48:40 +00:00
|
|
|
unsigned Size = 0;
|
|
|
|
|
2009-03-10 18:00:05 +00:00
|
|
|
for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
|
2009-03-12 23:18:09 +00:00
|
|
|
if (isa<DbgInfoIntrinsic>(BBI))
|
|
|
|
continue;
|
2005-09-20 01:48:40 +00:00
|
|
|
if (Size > 10) return false; // Don't clone large BB's.
|
2009-03-12 23:18:09 +00:00
|
|
|
++Size;
|
2005-09-20 00:43:16 +00:00
|
|
|
|
2009-03-12 23:18:09 +00:00
|
|
|
// We can only support instructions that do not define values that are
|
2005-09-20 01:48:40 +00:00
|
|
|
// live outside of the current basic block.
|
|
|
|
for (Value::use_iterator UI = BBI->use_begin(), E = BBI->use_end();
|
|
|
|
UI != E; ++UI) {
|
|
|
|
Instruction *U = cast<Instruction>(*UI);
|
|
|
|
if (U->getParent() != BB || isa<PHINode>(U)) return false;
|
|
|
|
}
|
2005-09-20 00:43:16 +00:00
|
|
|
|
|
|
|
// Looks ok, continue checking.
|
|
|
|
}
|
2005-09-20 01:48:40 +00:00
|
|
|
|
2005-09-20 00:43:16 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2005-09-19 23:49:37 +00:00
|
|
|
/// FoldCondBranchOnPHI - If we have a conditional branch on a PHI node value
|
|
|
|
/// that is defined in the same block as the branch and if any PHI entries are
|
|
|
|
/// constants, thread edges corresponding to that entry to be branches to their
|
|
|
|
/// ultimate destination.
|
2010-12-14 06:17:25 +00:00
|
|
|
static bool FoldCondBranchOnPHI(BranchInst *BI, const TargetData *TD) {
|
2005-09-19 23:49:37 +00:00
|
|
|
BasicBlock *BB = BI->getParent();
|
|
|
|
PHINode *PN = dyn_cast<PHINode>(BI->getCondition());
|
2005-09-19 23:57:04 +00:00
|
|
|
// NOTE: we currently cannot transform this case if the PHI node is used
|
|
|
|
// outside of the block.
|
2005-09-20 00:43:16 +00:00
|
|
|
if (!PN || PN->getParent() != BB || !PN->hasOneUse())
|
|
|
|
return false;
|
2005-09-19 23:49:37 +00:00
|
|
|
|
|
|
|
// Degenerate case of a single entry PHI.
|
|
|
|
if (PN->getNumIncomingValues() == 1) {
|
2008-12-03 19:44:02 +00:00
|
|
|
FoldSingleEntryPHINodes(PN->getParent());
|
2005-09-19 23:49:37 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Now we know that this block has multiple preds and two succs.
|
2005-09-20 00:43:16 +00:00
|
|
|
if (!BlockIsSimpleEnoughToThreadThrough(BB)) return false;
|
2005-09-19 23:49:37 +00:00
|
|
|
|
|
|
|
// Okay, this is a simple enough basic block. See if any phi values are
|
|
|
|
// constants.
|
2007-01-11 12:24:14 +00:00
|
|
|
for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
|
2010-12-13 01:47:07 +00:00
|
|
|
ConstantInt *CB = dyn_cast<ConstantInt>(PN->getIncomingValue(i));
|
|
|
|
if (CB == 0 || !CB->getType()->isIntegerTy(1)) continue;
|
|
|
|
|
|
|
|
// Okay, we now know that all edges from PredBB should be revectored to
|
|
|
|
// branch to RealDest.
|
|
|
|
BasicBlock *PredBB = PN->getIncomingBlock(i);
|
|
|
|
BasicBlock *RealDest = BI->getSuccessor(!CB->getZExtValue());
|
|
|
|
|
|
|
|
if (RealDest == BB) continue; // Skip self loops.
|
|
|
|
|
|
|
|
// The dest block might have PHI nodes, other predecessors and other
|
|
|
|
// difficult cases. Instead of being smart about this, just insert a new
|
|
|
|
// block that jumps to the destination block, effectively splitting
|
|
|
|
// the edge we are about to create.
|
|
|
|
BasicBlock *EdgeBB = BasicBlock::Create(BB->getContext(),
|
|
|
|
RealDest->getName()+".critedge",
|
|
|
|
RealDest->getParent(), RealDest);
|
|
|
|
BranchInst::Create(RealDest, EdgeBB);
|
2010-12-14 07:09:42 +00:00
|
|
|
|
|
|
|
// Update PHI nodes.
|
|
|
|
AddPredecessorToBlock(RealDest, EdgeBB, BB);
|
2010-12-13 01:47:07 +00:00
|
|
|
|
|
|
|
// BB may have instructions that are being threaded over. Clone these
|
|
|
|
// instructions into EdgeBB. We know that there will be no uses of the
|
|
|
|
// cloned instructions outside of EdgeBB.
|
|
|
|
BasicBlock::iterator InsertPt = EdgeBB->begin();
|
|
|
|
DenseMap<Value*, Value*> TranslateMap; // Track translated values.
|
|
|
|
for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
|
|
|
|
if (PHINode *PN = dyn_cast<PHINode>(BBI)) {
|
|
|
|
TranslateMap[PN] = PN->getIncomingValueForBlock(PredBB);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
// Clone the instruction.
|
|
|
|
Instruction *N = BBI->clone();
|
|
|
|
if (BBI->hasName()) N->setName(BBI->getName()+".c");
|
2005-09-19 23:49:37 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Update operands due to translation.
|
|
|
|
for (User::op_iterator i = N->op_begin(), e = N->op_end();
|
|
|
|
i != e; ++i) {
|
|
|
|
DenseMap<Value*, Value*>::iterator PI = TranslateMap.find(*i);
|
|
|
|
if (PI != TranslateMap.end())
|
|
|
|
*i = PI->second;
|
|
|
|
}
|
2005-09-19 23:49:37 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Check for trivial simplification.
|
2010-12-14 06:17:25 +00:00
|
|
|
if (Value *V = SimplifyInstruction(N, TD)) {
|
|
|
|
TranslateMap[BBI] = V;
|
|
|
|
delete N; // Instruction folded away, don't need actual inst
|
2010-12-13 01:47:07 +00:00
|
|
|
} else {
|
|
|
|
// Insert the new instruction into its new home.
|
|
|
|
EdgeBB->getInstList().insert(InsertPt, N);
|
|
|
|
if (!BBI->use_empty())
|
|
|
|
TranslateMap[BBI] = N;
|
2005-09-20 01:48:40 +00:00
|
|
|
}
|
2010-12-13 01:47:07 +00:00
|
|
|
}
|
2005-09-20 01:48:40 +00:00
|
|
|
|
2010-12-13 01:47:07 +00:00
|
|
|
// Loop over all of the edges from PredBB to BB, changing them to branch
|
|
|
|
// to EdgeBB instead.
|
|
|
|
TerminatorInst *PredBBTI = PredBB->getTerminator();
|
|
|
|
for (unsigned i = 0, e = PredBBTI->getNumSuccessors(); i != e; ++i)
|
|
|
|
if (PredBBTI->getSuccessor(i) == BB) {
|
|
|
|
BB->removePredecessor(PredBB);
|
|
|
|
PredBBTI->setSuccessor(i, EdgeBB);
|
2005-09-20 01:48:40 +00:00
|
|
|
}
|
2010-12-13 01:47:07 +00:00
|
|
|
|
|
|
|
// Recurse, simplifying any other constants.
|
2010-12-14 06:17:25 +00:00
|
|
|
return FoldCondBranchOnPHI(BI, TD) | true;
|
2007-01-11 12:24:14 +00:00
|
|
|
}
|
2005-09-19 23:49:37 +00:00
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2005-09-23 06:39:30 +00:00
|
|
|
/// FoldTwoEntryPHINode - Given a BB that starts with the specified two-entry
|
|
|
|
/// PHI node, see if we can eliminate it.
|
2010-12-14 07:00:00 +00:00
|
|
|
static bool FoldTwoEntryPHINode(PHINode *PN, const TargetData *TD) {
|
2005-09-23 06:39:30 +00:00
|
|
|
// Ok, this is a two entry PHI node. Check to see if this is a simple "if
|
|
|
|
// statement", which has a very simple dominance structure. Basically, we
|
|
|
|
// are trying to find the condition that is being branched on, which
|
|
|
|
// subsequently causes this merge to happen. We really want control
|
|
|
|
// dependence information for this check, but simplifycfg can't keep it up
|
|
|
|
// to date, and this catches most of the cases we care about anyway.
|
|
|
|
BasicBlock *BB = PN->getParent();
|
|
|
|
BasicBlock *IfTrue, *IfFalse;
|
|
|
|
Value *IfCond = GetIfCondition(BB, IfTrue, IfFalse);
|
2010-12-14 08:01:53 +00:00
|
|
|
if (!IfCond ||
|
|
|
|
// Don't bother if the branch will be constant folded trivially.
|
|
|
|
isa<ConstantInt>(IfCond))
|
|
|
|
return false;
|
2005-09-23 06:39:30 +00:00
|
|
|
|
2006-11-18 19:19:36 +00:00
|
|
|
// Okay, we found that we can merge this two-entry phi node into a select.
|
|
|
|
// Doing so would require us to fold *all* two entry phi nodes in this block.
|
|
|
|
// At some point this becomes non-profitable (particularly if the target
|
|
|
|
// doesn't support cmov's). Only do this transformation if there are two or
|
|
|
|
// fewer PHI nodes in this block.
|
|
|
|
unsigned NumPhis = 0;
|
|
|
|
for (BasicBlock::iterator I = BB->begin(); isa<PHINode>(I); ++NumPhis, ++I)
|
|
|
|
if (NumPhis > 2)
|
|
|
|
return false;
|
|
|
|
|
2005-09-23 06:39:30 +00:00
|
|
|
// Loop over the PHI's seeing if we can promote them all to select
|
|
|
|
// instructions. While we are at it, keep track of the instructions
|
|
|
|
// that need to be moved to the dominating block.
|
2010-12-14 07:41:39 +00:00
|
|
|
SmallPtrSet<Instruction*, 4> AggressiveInsts;
|
2005-09-23 06:39:30 +00:00
|
|
|
|
2010-12-14 08:46:09 +00:00
|
|
|
for (BasicBlock::iterator II = BB->begin(); isa<PHINode>(II);) {
|
|
|
|
PHINode *PN = cast<PHINode>(II++);
|
2010-12-14 07:20:29 +00:00
|
|
|
if (Value *V = SimplifyInstruction(PN, TD)) {
|
|
|
|
PN->replaceAllUsesWith(V);
|
2010-12-14 08:46:09 +00:00
|
|
|
PN->eraseFromParent();
|
2010-12-14 07:20:29 +00:00
|
|
|
continue;
|
2005-09-23 06:39:30 +00:00
|
|
|
}
|
2010-12-14 07:20:29 +00:00
|
|
|
|
|
|
|
if (!DominatesMergePoint(PN->getIncomingValue(0), BB, &AggressiveInsts) ||
|
|
|
|
!DominatesMergePoint(PN->getIncomingValue(1), BB, &AggressiveInsts))
|
|
|
|
return false;
|
2005-09-23 06:39:30 +00:00
|
|
|
}
|
|
|
|
|
2010-12-14 07:41:39 +00:00
|
|
|
// If we folded the the first phi, PN dangles at this point. Refresh it. If
|
|
|
|
// we ran out of PHIs then we simplified them all.
|
|
|
|
PN = dyn_cast<PHINode>(BB->begin());
|
|
|
|
if (PN == 0) return true;
|
|
|
|
|
2010-12-14 08:46:09 +00:00
|
|
|
// Don't fold i1 branches on PHIs which contain binary operators. These can
|
|
|
|
// often be turned into switches and other things.
|
|
|
|
if (PN->getType()->isIntegerTy(1) &&
|
|
|
|
(isa<BinaryOperator>(PN->getIncomingValue(0)) ||
|
|
|
|
isa<BinaryOperator>(PN->getIncomingValue(1)) ||
|
|
|
|
isa<BinaryOperator>(IfCond)))
|
|
|
|
return false;
|
|
|
|
|
2005-09-23 06:39:30 +00:00
|
|
|
// If we all PHI nodes are promotable, check to make sure that all
|
|
|
|
// instructions in the predecessor blocks can be promoted as well. If
|
|
|
|
// not, we won't be able to get rid of the control flow, so it's not
|
|
|
|
// worth promoting to select instructions.
|
2010-12-14 07:41:39 +00:00
|
|
|
BasicBlock *DomBlock = 0;
|
|
|
|
BasicBlock *IfBlock1 = PN->getIncomingBlock(0);
|
|
|
|
BasicBlock *IfBlock2 = PN->getIncomingBlock(1);
|
|
|
|
if (cast<BranchInst>(IfBlock1->getTerminator())->isConditional()) {
|
|
|
|
IfBlock1 = 0;
|
|
|
|
} else {
|
|
|
|
DomBlock = *pred_begin(IfBlock1);
|
|
|
|
for (BasicBlock::iterator I = IfBlock1->begin();!isa<TerminatorInst>(I);++I)
|
2009-02-03 22:12:02 +00:00
|
|
|
if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
|
2005-09-23 06:39:30 +00:00
|
|
|
// This is not an aggressive instruction that we can promote.
|
|
|
|
// Because of this, we won't be able to get rid of the control
|
|
|
|
// flow, so the xform is not worth it.
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-12-14 07:41:39 +00:00
|
|
|
if (cast<BranchInst>(IfBlock2->getTerminator())->isConditional()) {
|
|
|
|
IfBlock2 = 0;
|
|
|
|
} else {
|
|
|
|
DomBlock = *pred_begin(IfBlock2);
|
|
|
|
for (BasicBlock::iterator I = IfBlock2->begin();!isa<TerminatorInst>(I);++I)
|
2009-02-03 22:12:02 +00:00
|
|
|
if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
|
2005-09-23 06:39:30 +00:00
|
|
|
// This is not an aggressive instruction that we can promote.
|
|
|
|
// Because of this, we won't be able to get rid of the control
|
|
|
|
// flow, so the xform is not worth it.
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
2010-12-14 07:23:10 +00:00
|
|
|
|
|
|
|
DEBUG(dbgs() << "FOUND IF CONDITION! " << *IfCond << " T: "
|
2010-12-14 07:41:39 +00:00
|
|
|
<< IfTrue->getName() << " F: " << IfFalse->getName() << "\n");
|
2005-09-23 06:39:30 +00:00
|
|
|
|
|
|
|
// If we can still promote the PHI nodes after this gauntlet of tests,
|
|
|
|
// do all of the PHI's now.
|
2010-12-14 08:46:09 +00:00
|
|
|
Instruction *InsertPt = DomBlock->getTerminator();
|
|
|
|
|
2005-09-23 06:39:30 +00:00
|
|
|
// Move all 'aggressive' instructions, which are defined in the
|
|
|
|
// conditional parts of the if's up to the dominating block.
|
2010-12-13 01:47:07 +00:00
|
|
|
if (IfBlock1)
|
2010-12-14 08:46:09 +00:00
|
|
|
DomBlock->getInstList().splice(InsertPt,
|
2010-12-13 01:47:07 +00:00
|
|
|
IfBlock1->getInstList(), IfBlock1->begin(),
|
2005-09-23 06:39:30 +00:00
|
|
|
IfBlock1->getTerminator());
|
2010-12-13 01:47:07 +00:00
|
|
|
if (IfBlock2)
|
2010-12-14 08:46:09 +00:00
|
|
|
DomBlock->getInstList().splice(InsertPt,
|
2010-12-13 01:47:07 +00:00
|
|
|
IfBlock2->getInstList(), IfBlock2->begin(),
|
2005-09-23 06:39:30 +00:00
|
|
|
IfBlock2->getTerminator());
|
|
|
|
|
|
|
|
while (PHINode *PN = dyn_cast<PHINode>(BB->begin())) {
|
|
|
|
// Change the PHI node into a select instruction.
|
2010-12-13 01:47:07 +00:00
|
|
|
Value *TrueVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfFalse);
|
|
|
|
Value *FalseVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfTrue);
|
2005-09-23 06:39:30 +00:00
|
|
|
|
2010-12-14 08:46:09 +00:00
|
|
|
Value *NV = SelectInst::Create(IfCond, TrueVal, FalseVal, "", InsertPt);
|
2007-02-11 01:37:51 +00:00
|
|
|
PN->replaceAllUsesWith(NV);
|
|
|
|
NV->takeName(PN);
|
2010-12-14 06:17:25 +00:00
|
|
|
PN->eraseFromParent();
|
2005-09-23 06:39:30 +00:00
|
|
|
}
|
2010-12-14 08:01:53 +00:00
|
|
|
|
|
|
|
// At this point, IfBlock1 and IfBlock2 are both empty, so our if statement
|
|
|
|
// has been flattened. Change DomBlock to jump directly to our new block to
|
|
|
|
// avoid other simplifycfg's kicking in on the diamond.
|
|
|
|
TerminatorInst *OldTI = DomBlock->getTerminator();
|
|
|
|
BranchInst::Create(BB, OldTI);
|
|
|
|
OldTI->eraseFromParent();
|
2005-09-23 06:39:30 +00:00
|
|
|
return true;
|
|
|
|
}
|
2005-09-19 23:49:37 +00:00
|
|
|
|
2008-04-24 00:01:19 +00:00
|
|
|
/// SimplifyCondBranchToTwoReturns - If we found a conditional branch that goes
|
|
|
|
/// to two returning blocks, try to merge them together into one return,
|
|
|
|
/// introducing a select if the return values disagree.
|
|
|
|
static bool SimplifyCondBranchToTwoReturns(BranchInst *BI) {
|
|
|
|
assert(BI->isConditional() && "Must be a conditional branch");
|
|
|
|
BasicBlock *TrueSucc = BI->getSuccessor(0);
|
|
|
|
BasicBlock *FalseSucc = BI->getSuccessor(1);
|
|
|
|
ReturnInst *TrueRet = cast<ReturnInst>(TrueSucc->getTerminator());
|
|
|
|
ReturnInst *FalseRet = cast<ReturnInst>(FalseSucc->getTerminator());
|
|
|
|
|
|
|
|
// Check to ensure both blocks are empty (just a return) or optionally empty
|
|
|
|
// with PHI nodes. If there are other instructions, merging would cause extra
|
|
|
|
// computation on one path or the other.
|
2010-12-13 01:47:07 +00:00
|
|
|
if (!TrueSucc->getFirstNonPHIOrDbg()->isTerminator())
|
2009-02-05 00:30:42 +00:00
|
|
|
return false;
|
2010-12-13 01:47:07 +00:00
|
|
|
if (!FalseSucc->getFirstNonPHIOrDbg()->isTerminator())
|
2009-02-05 00:30:42 +00:00
|
|
|
return false;
|
2008-04-24 00:01:19 +00:00
|
|
|
|
|
|
|
// Okay, we found a branch that is going to two return nodes. If
|
|
|
|
// there is no return value for this function, just change the
|
|
|
|
// branch into a return.
|
|
|
|
if (FalseRet->getNumOperands() == 0) {
|
|
|
|
TrueSucc->removePredecessor(BI->getParent());
|
|
|
|
FalseSucc->removePredecessor(BI->getParent());
|
2009-08-13 21:58:54 +00:00
|
|
|
ReturnInst::Create(BI->getContext(), 0, BI);
|
2008-12-16 20:54:32 +00:00
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
2008-04-24 00:01:19 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2008-07-23 00:34:11 +00:00
|
|
|
// Otherwise, figure out what the true and false return values are
|
|
|
|
// so we can insert a new select instruction.
|
|
|
|
Value *TrueValue = TrueRet->getReturnValue();
|
|
|
|
Value *FalseValue = FalseRet->getReturnValue();
|
|
|
|
|
|
|
|
// Unwrap any PHI nodes in the return blocks.
|
|
|
|
if (PHINode *TVPN = dyn_cast_or_null<PHINode>(TrueValue))
|
|
|
|
if (TVPN->getParent() == TrueSucc)
|
|
|
|
TrueValue = TVPN->getIncomingValueForBlock(BI->getParent());
|
|
|
|
if (PHINode *FVPN = dyn_cast_or_null<PHINode>(FalseValue))
|
|
|
|
if (FVPN->getParent() == FalseSucc)
|
|
|
|
FalseValue = FVPN->getIncomingValueForBlock(BI->getParent());
|
|
|
|
|
|
|
|
// In order for this transformation to be safe, we must be able to
|
|
|
|
// unconditionally execute both operands to the return. This is
|
|
|
|
// normally the case, but we could have a potentially-trapping
|
|
|
|
// constant expression that prevents this transformation from being
|
|
|
|
// safe.
|
|
|
|
if (ConstantExpr *TCV = dyn_cast_or_null<ConstantExpr>(TrueValue))
|
|
|
|
if (TCV->canTrap())
|
|
|
|
return false;
|
|
|
|
if (ConstantExpr *FCV = dyn_cast_or_null<ConstantExpr>(FalseValue))
|
|
|
|
if (FCV->canTrap())
|
|
|
|
return false;
|
2008-04-24 00:01:19 +00:00
|
|
|
|
|
|
|
// Okay, we collected all the mapped values and checked them for sanity, and
|
|
|
|
// defined to really do this transformation. First, update the CFG.
|
|
|
|
TrueSucc->removePredecessor(BI->getParent());
|
|
|
|
FalseSucc->removePredecessor(BI->getParent());
|
|
|
|
|
|
|
|
// Insert select instructions where needed.
|
|
|
|
Value *BrCond = BI->getCondition();
|
2008-07-23 00:34:11 +00:00
|
|
|
if (TrueValue) {
|
2008-04-24 00:01:19 +00:00
|
|
|
// Insert a select if the results differ.
|
2008-07-23 00:34:11 +00:00
|
|
|
if (TrueValue == FalseValue || isa<UndefValue>(FalseValue)) {
|
|
|
|
} else if (isa<UndefValue>(TrueValue)) {
|
|
|
|
TrueValue = FalseValue;
|
|
|
|
} else {
|
|
|
|
TrueValue = SelectInst::Create(BrCond, TrueValue,
|
|
|
|
FalseValue, "retval", BI);
|
2008-04-24 00:01:19 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-07-23 00:34:11 +00:00
|
|
|
Value *RI = !TrueValue ?
|
2009-08-13 21:58:54 +00:00
|
|
|
ReturnInst::Create(BI->getContext(), BI) :
|
|
|
|
ReturnInst::Create(BI->getContext(), TrueValue, BI);
|
2009-08-23 10:29:55 +00:00
|
|
|
(void) RI;
|
2008-04-24 00:01:19 +00:00
|
|
|
|
2010-01-05 01:26:52 +00:00
|
|
|
DEBUG(dbgs() << "\nCHANGING BRANCH TO TWO RETURNS INTO SELECT:"
|
2009-08-23 04:37:46 +00:00
|
|
|
<< "\n " << *BI << "NewRet = " << *RI
|
|
|
|
<< "TRUEBLOCK: " << *TrueSucc << "FALSEBLOCK: "<< *FalseSucc);
|
2008-04-24 00:01:19 +00:00
|
|
|
|
2008-12-16 20:54:32 +00:00
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
|
|
|
|
2008-04-24 00:01:19 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2008-07-13 21:12:01 +00:00
|
|
|
/// FoldBranchToCommonDest - If this basic block is ONLY a setcc and a branch,
|
|
|
|
/// and if a predecessor branches to us and one of our successors, fold the
|
|
|
|
/// setcc into the predecessor and use logical operations to pick the right
|
|
|
|
/// destination.
|
2009-06-27 21:30:38 +00:00
|
|
|
bool llvm::FoldBranchToCommonDest(BranchInst *BI) {
|
2008-07-13 22:23:11 +00:00
|
|
|
BasicBlock *BB = BI->getParent();
|
2008-07-13 21:12:01 +00:00
|
|
|
Instruction *Cond = dyn_cast<Instruction>(BI->getCondition());
|
2010-07-14 19:52:16 +00:00
|
|
|
if (Cond == 0 || (!isa<CmpInst>(Cond) && !isa<BinaryOperator>(Cond)) ||
|
|
|
|
Cond->getParent() != BB || !Cond->hasOneUse())
|
|
|
|
return false;
|
2008-07-13 22:23:11 +00:00
|
|
|
|
2008-07-13 21:12:01 +00:00
|
|
|
// Only allow this if the condition is a simple instruction that can be
|
|
|
|
// executed unconditionally. It must be in the same block as the branch, and
|
|
|
|
// must be at the front of the block.
|
2009-02-04 21:39:48 +00:00
|
|
|
BasicBlock::iterator FrontIt = BB->front();
|
|
|
|
// Ignore dbg intrinsics.
|
Make simplifycfg reprocess newly formed "br (cond1 | cond2)" conditions
when simplifying, allowing them to be eagerly turned into switches. This
is the last step required to get "Example 7" from this blog post:
http://blog.regehr.org/archives/320
On X86, we now generate this machine code, which (to my eye) seems better
than the ICC generated code:
_crud: ## @crud
## BB#0: ## %entry
cmpb $33, %dil
jb LBB0_4
## BB#1: ## %switch.early.test
addb $-34, %dil
cmpb $58, %dil
ja LBB0_3
## BB#2: ## %switch.early.test
movzbl %dil, %eax
movabsq $288230376537592865, %rcx ## imm = 0x400000017001421
btq %rax, %rcx
jb LBB0_4
LBB0_3: ## %lor.rhs
xorl %eax, %eax
ret
LBB0_4: ## %lor.end
movl $1, %eax
ret
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121690 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 07:00:06 +00:00
|
|
|
while (isa<DbgInfoIntrinsic>(FrontIt))
|
2009-02-04 21:39:48 +00:00
|
|
|
++FrontIt;
|
2010-07-14 19:52:16 +00:00
|
|
|
|
|
|
|
// Allow a single instruction to be hoisted in addition to the compare
|
|
|
|
// that feeds the branch. We later ensure that any values that _it_ uses
|
|
|
|
// were also live in the predecessor, so that we don't unnecessarily create
|
|
|
|
// register pressure or inhibit out-of-order execution.
|
|
|
|
Instruction *BonusInst = 0;
|
|
|
|
if (&*FrontIt != Cond &&
|
2010-07-15 16:38:22 +00:00
|
|
|
FrontIt->hasOneUse() && *FrontIt->use_begin() == Cond &&
|
|
|
|
FrontIt->isSafeToSpeculativelyExecute()) {
|
2010-07-14 19:52:16 +00:00
|
|
|
BonusInst = &*FrontIt;
|
|
|
|
++FrontIt;
|
2009-02-04 21:39:48 +00:00
|
|
|
}
|
2009-01-19 23:03:13 +00:00
|
|
|
|
2010-07-14 19:52:16 +00:00
|
|
|
// Only a single bonus inst is allowed.
|
|
|
|
if (&*FrontIt != Cond)
|
|
|
|
return false;
|
|
|
|
|
2008-07-13 21:12:01 +00:00
|
|
|
// Make sure the instruction after the condition is the cond branch.
|
|
|
|
BasicBlock::iterator CondIt = Cond; ++CondIt;
|
2009-02-04 21:39:48 +00:00
|
|
|
// Ingore dbg intrinsics.
|
|
|
|
while(isa<DbgInfoIntrinsic>(CondIt))
|
|
|
|
++CondIt;
|
|
|
|
if (&*CondIt != BI) {
|
|
|
|
assert (!isa<DbgInfoIntrinsic>(CondIt) && "Hey do not forget debug info!");
|
2008-07-13 21:12:01 +00:00
|
|
|
return false;
|
2009-02-04 21:39:48 +00:00
|
|
|
}
|
2009-01-19 23:03:13 +00:00
|
|
|
|
|
|
|
// Cond is known to be a compare or binary operator. Check to make sure that
|
|
|
|
// neither operand is a potentially-trapping constant expression.
|
|
|
|
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(0)))
|
|
|
|
if (CE->canTrap())
|
|
|
|
return false;
|
|
|
|
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(1)))
|
|
|
|
if (CE->canTrap())
|
|
|
|
return false;
|
|
|
|
|
2008-07-13 21:12:01 +00:00
|
|
|
|
|
|
|
// Finally, don't infinitely unroll conditional loops.
|
|
|
|
BasicBlock *TrueDest = BI->getSuccessor(0);
|
|
|
|
BasicBlock *FalseDest = BI->getSuccessor(1);
|
|
|
|
if (TrueDest == BB || FalseDest == BB)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
|
|
|
|
BasicBlock *PredBlock = *PI;
|
|
|
|
BranchInst *PBI = dyn_cast<BranchInst>(PredBlock->getTerminator());
|
2009-01-19 23:03:13 +00:00
|
|
|
|
2008-07-13 22:23:11 +00:00
|
|
|
// Check that we have two conditional branches. If there is a PHI node in
|
|
|
|
// the common successor, verify that the same value flows in from both
|
|
|
|
// blocks.
|
2008-07-13 21:12:01 +00:00
|
|
|
if (PBI == 0 || PBI->isUnconditional() ||
|
|
|
|
!SafeToMergeTerminators(BI, PBI))
|
|
|
|
continue;
|
|
|
|
|
2010-07-14 19:52:16 +00:00
|
|
|
// Ensure that any values used in the bonus instruction are also used
|
|
|
|
// by the terminator of the predecessor. This means that those values
|
|
|
|
// must already have been resolved, so we won't be inhibiting the
|
|
|
|
// out-of-order core by speculating them earlier.
|
|
|
|
if (BonusInst) {
|
|
|
|
// Collect the values used by the bonus inst
|
|
|
|
SmallPtrSet<Value*, 4> UsedValues;
|
|
|
|
for (Instruction::op_iterator OI = BonusInst->op_begin(),
|
|
|
|
OE = BonusInst->op_end(); OI != OE; ++OI) {
|
|
|
|
Value* V = *OI;
|
|
|
|
if (!isa<Constant>(V))
|
|
|
|
UsedValues.insert(V);
|
|
|
|
}
|
|
|
|
|
|
|
|
SmallVector<std::pair<Value*, unsigned>, 4> Worklist;
|
|
|
|
Worklist.push_back(std::make_pair(PBI->getOperand(0), 0));
|
|
|
|
|
|
|
|
// Walk up to four levels back up the use-def chain of the predecessor's
|
|
|
|
// terminator to see if all those values were used. The choice of four
|
|
|
|
// levels is arbitrary, to provide a compile-time-cost bound.
|
|
|
|
while (!Worklist.empty()) {
|
|
|
|
std::pair<Value*, unsigned> Pair = Worklist.back();
|
|
|
|
Worklist.pop_back();
|
|
|
|
|
|
|
|
if (Pair.second >= 4) continue;
|
|
|
|
UsedValues.erase(Pair.first);
|
|
|
|
if (UsedValues.empty()) break;
|
|
|
|
|
Make simplifycfg reprocess newly formed "br (cond1 | cond2)" conditions
when simplifying, allowing them to be eagerly turned into switches. This
is the last step required to get "Example 7" from this blog post:
http://blog.regehr.org/archives/320
On X86, we now generate this machine code, which (to my eye) seems better
than the ICC generated code:
_crud: ## @crud
## BB#0: ## %entry
cmpb $33, %dil
jb LBB0_4
## BB#1: ## %switch.early.test
addb $-34, %dil
cmpb $58, %dil
ja LBB0_3
## BB#2: ## %switch.early.test
movzbl %dil, %eax
movabsq $288230376537592865, %rcx ## imm = 0x400000017001421
btq %rax, %rcx
jb LBB0_4
LBB0_3: ## %lor.rhs
xorl %eax, %eax
ret
LBB0_4: ## %lor.end
movl $1, %eax
ret
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121690 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 07:00:06 +00:00
|
|
|
if (Instruction *I = dyn_cast<Instruction>(Pair.first)) {
|
2010-07-14 19:52:16 +00:00
|
|
|
for (Instruction::op_iterator OI = I->op_begin(), OE = I->op_end();
|
|
|
|
OI != OE; ++OI)
|
|
|
|
Worklist.push_back(std::make_pair(OI->get(), Pair.second+1));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!UsedValues.empty()) return false;
|
|
|
|
}
|
|
|
|
|
2008-07-13 21:20:19 +00:00
|
|
|
Instruction::BinaryOps Opc;
|
|
|
|
bool InvertPredCond = false;
|
|
|
|
|
|
|
|
if (PBI->getSuccessor(0) == TrueDest)
|
|
|
|
Opc = Instruction::Or;
|
|
|
|
else if (PBI->getSuccessor(1) == FalseDest)
|
|
|
|
Opc = Instruction::And;
|
|
|
|
else if (PBI->getSuccessor(0) == FalseDest)
|
|
|
|
Opc = Instruction::And, InvertPredCond = true;
|
|
|
|
else if (PBI->getSuccessor(1) == TrueDest)
|
|
|
|
Opc = Instruction::Or, InvertPredCond = true;
|
|
|
|
else
|
|
|
|
continue;
|
|
|
|
|
2010-01-05 01:26:52 +00:00
|
|
|
DEBUG(dbgs() << "FOLDING BRANCH TO COMMON DEST:\n" << *PBI << *BB);
|
2009-01-19 23:03:13 +00:00
|
|
|
|
2008-07-13 21:20:19 +00:00
|
|
|
// If we need to invert the condition in the pred block to match, do so now.
|
|
|
|
if (InvertPredCond) {
|
Make simplifycfg reprocess newly formed "br (cond1 | cond2)" conditions
when simplifying, allowing them to be eagerly turned into switches. This
is the last step required to get "Example 7" from this blog post:
http://blog.regehr.org/archives/320
On X86, we now generate this machine code, which (to my eye) seems better
than the ICC generated code:
_crud: ## @crud
## BB#0: ## %entry
cmpb $33, %dil
jb LBB0_4
## BB#1: ## %switch.early.test
addb $-34, %dil
cmpb $58, %dil
ja LBB0_3
## BB#2: ## %switch.early.test
movzbl %dil, %eax
movabsq $288230376537592865, %rcx ## imm = 0x400000017001421
btq %rax, %rcx
jb LBB0_4
LBB0_3: ## %lor.rhs
xorl %eax, %eax
ret
LBB0_4: ## %lor.end
movl $1, %eax
ret
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121690 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 07:00:06 +00:00
|
|
|
Value *NewCond = PBI->getCondition();
|
|
|
|
|
|
|
|
if (NewCond->hasOneUse() && isa<CmpInst>(NewCond)) {
|
|
|
|
CmpInst *CI = cast<CmpInst>(NewCond);
|
|
|
|
CI->setPredicate(CI->getInversePredicate());
|
|
|
|
} else {
|
|
|
|
NewCond = BinaryOperator::CreateNot(NewCond,
|
2008-07-13 21:20:19 +00:00
|
|
|
PBI->getCondition()->getName()+".not", PBI);
|
Make simplifycfg reprocess newly formed "br (cond1 | cond2)" conditions
when simplifying, allowing them to be eagerly turned into switches. This
is the last step required to get "Example 7" from this blog post:
http://blog.regehr.org/archives/320
On X86, we now generate this machine code, which (to my eye) seems better
than the ICC generated code:
_crud: ## @crud
## BB#0: ## %entry
cmpb $33, %dil
jb LBB0_4
## BB#1: ## %switch.early.test
addb $-34, %dil
cmpb $58, %dil
ja LBB0_3
## BB#2: ## %switch.early.test
movzbl %dil, %eax
movabsq $288230376537592865, %rcx ## imm = 0x400000017001421
btq %rax, %rcx
jb LBB0_4
LBB0_3: ## %lor.rhs
xorl %eax, %eax
ret
LBB0_4: ## %lor.end
movl $1, %eax
ret
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121690 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 07:00:06 +00:00
|
|
|
}
|
|
|
|
|
2008-07-13 21:12:01 +00:00
|
|
|
PBI->setCondition(NewCond);
|
|
|
|
BasicBlock *OldTrue = PBI->getSuccessor(0);
|
|
|
|
BasicBlock *OldFalse = PBI->getSuccessor(1);
|
|
|
|
PBI->setSuccessor(0, OldFalse);
|
|
|
|
PBI->setSuccessor(1, OldTrue);
|
|
|
|
}
|
2008-07-13 21:15:11 +00:00
|
|
|
|
2010-07-14 19:52:16 +00:00
|
|
|
// If we have a bonus inst, clone it into the predecessor block.
|
|
|
|
Instruction *NewBonus = 0;
|
|
|
|
if (BonusInst) {
|
|
|
|
NewBonus = BonusInst->clone();
|
|
|
|
PredBlock->getInstList().insert(PBI, NewBonus);
|
|
|
|
NewBonus->takeName(BonusInst);
|
|
|
|
BonusInst->setName(BonusInst->getName()+".old");
|
|
|
|
}
|
|
|
|
|
2008-07-13 21:20:19 +00:00
|
|
|
// Clone Cond into the predecessor basic block, and or/and the
|
|
|
|
// two conditions together.
|
2009-09-27 07:38:41 +00:00
|
|
|
Instruction *New = Cond->clone();
|
2010-07-14 19:52:16 +00:00
|
|
|
if (BonusInst) New->replaceUsesOfWith(BonusInst, NewBonus);
|
2008-07-13 21:20:19 +00:00
|
|
|
PredBlock->getInstList().insert(PBI, New);
|
|
|
|
New->takeName(Cond);
|
|
|
|
Cond->setName(New->getName()+".old");
|
2008-07-13 21:15:11 +00:00
|
|
|
|
2008-07-13 21:20:19 +00:00
|
|
|
Value *NewCond = BinaryOperator::Create(Opc, PBI->getCondition(),
|
|
|
|
New, "or.cond", PBI);
|
|
|
|
PBI->setCondition(NewCond);
|
|
|
|
if (PBI->getSuccessor(0) == BB) {
|
|
|
|
AddPredecessorToBlock(TrueDest, PredBlock, BB);
|
|
|
|
PBI->setSuccessor(0, TrueDest);
|
|
|
|
}
|
|
|
|
if (PBI->getSuccessor(1) == BB) {
|
|
|
|
AddPredecessorToBlock(FalseDest, PredBlock, BB);
|
|
|
|
PBI->setSuccessor(1, FalseDest);
|
2008-07-13 21:12:01 +00:00
|
|
|
}
|
2010-12-14 05:57:30 +00:00
|
|
|
return true;
|
2008-07-13 21:12:01 +00:00
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2008-07-13 21:53:26 +00:00
|
|
|
/// SimplifyCondBranchToCondBranch - If we have a conditional branch as a
|
|
|
|
/// predecessor of another block, this function tries to simplify it. We know
|
|
|
|
/// that PBI and BI are both conditional branches, and BI is in one of the
|
|
|
|
/// successor blocks of PBI - PBI branches to BI.
|
|
|
|
static bool SimplifyCondBranchToCondBranch(BranchInst *PBI, BranchInst *BI) {
|
|
|
|
assert(PBI->isConditional() && BI->isConditional());
|
|
|
|
BasicBlock *BB = BI->getParent();
|
2009-08-12 16:23:25 +00:00
|
|
|
|
2008-07-13 21:53:26 +00:00
|
|
|
// If this block ends with a branch instruction, and if there is a
|
|
|
|
// predecessor that ends on a branch of the same condition, make
|
|
|
|
// this conditional branch redundant.
|
|
|
|
if (PBI->getCondition() == BI->getCondition() &&
|
|
|
|
PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
|
|
|
|
// Okay, the outcome of this conditional branch is statically
|
|
|
|
// knowable. If this block had a single pred, handle specially.
|
|
|
|
if (BB->getSinglePredecessor()) {
|
|
|
|
// Turn this into a branch on constant.
|
|
|
|
bool CondIsTrue = PBI->getSuccessor(0) == BB;
|
2009-08-13 21:58:54 +00:00
|
|
|
BI->setCondition(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
|
|
|
|
CondIsTrue));
|
2008-07-13 21:53:26 +00:00
|
|
|
return true; // Nuke the branch on constant.
|
|
|
|
}
|
|
|
|
|
|
|
|
// Otherwise, if there are multiple predecessors, insert a PHI that merges
|
|
|
|
// in the constant and simplify the block result. Subsequent passes of
|
|
|
|
// simplifycfg will thread the block.
|
|
|
|
if (BlockIsSimpleEnoughToThreadThrough(BB)) {
|
2009-08-13 21:58:54 +00:00
|
|
|
PHINode *NewPN = PHINode::Create(Type::getInt1Ty(BB->getContext()),
|
2008-07-13 21:53:26 +00:00
|
|
|
BI->getCondition()->getName() + ".pr",
|
|
|
|
BB->begin());
|
2008-07-13 21:55:46 +00:00
|
|
|
// Okay, we're going to insert the PHI node. Since PBI is not the only
|
|
|
|
// predecessor, compute the PHI'd conditional value for all of the preds.
|
|
|
|
// Any predecessor where the condition is not computable we keep symbolic.
|
2010-07-12 10:59:23 +00:00
|
|
|
for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
|
|
|
|
BasicBlock *P = *PI;
|
|
|
|
if ((PBI = dyn_cast<BranchInst>(P->getTerminator())) &&
|
2008-07-13 21:53:26 +00:00
|
|
|
PBI != BI && PBI->isConditional() &&
|
|
|
|
PBI->getCondition() == BI->getCondition() &&
|
|
|
|
PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
|
|
|
|
bool CondIsTrue = PBI->getSuccessor(0) == BB;
|
2009-08-13 21:58:54 +00:00
|
|
|
NewPN->addIncoming(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
|
2010-07-12 10:59:23 +00:00
|
|
|
CondIsTrue), P);
|
2008-07-13 21:53:26 +00:00
|
|
|
} else {
|
2010-07-12 10:59:23 +00:00
|
|
|
NewPN->addIncoming(BI->getCondition(), P);
|
2008-07-13 21:53:26 +00:00
|
|
|
}
|
2010-07-12 10:59:23 +00:00
|
|
|
}
|
2008-07-13 21:53:26 +00:00
|
|
|
|
|
|
|
BI->setCondition(NewPN);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// If this is a conditional branch in an empty block, and if any
|
|
|
|
// predecessors is a conditional branch to one of our destinations,
|
|
|
|
// fold the conditions into logical ops and one cond br.
|
2009-02-26 06:56:37 +00:00
|
|
|
BasicBlock::iterator BBI = BB->begin();
|
|
|
|
// Ignore dbg intrinsics.
|
|
|
|
while (isa<DbgInfoIntrinsic>(BBI))
|
|
|
|
++BBI;
|
|
|
|
if (&*BBI != BI)
|
2008-07-13 22:04:41 +00:00
|
|
|
return false;
|
2009-01-20 01:15:41 +00:00
|
|
|
|
|
|
|
|
|
|
|
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(BI->getCondition()))
|
|
|
|
if (CE->canTrap())
|
|
|
|
return false;
|
2008-07-13 22:04:41 +00:00
|
|
|
|
|
|
|
int PBIOp, BIOp;
|
|
|
|
if (PBI->getSuccessor(0) == BI->getSuccessor(0))
|
|
|
|
PBIOp = BIOp = 0;
|
|
|
|
else if (PBI->getSuccessor(0) == BI->getSuccessor(1))
|
|
|
|
PBIOp = 0, BIOp = 1;
|
|
|
|
else if (PBI->getSuccessor(1) == BI->getSuccessor(0))
|
|
|
|
PBIOp = 1, BIOp = 0;
|
|
|
|
else if (PBI->getSuccessor(1) == BI->getSuccessor(1))
|
|
|
|
PBIOp = BIOp = 1;
|
|
|
|
else
|
|
|
|
return false;
|
2008-07-13 21:53:26 +00:00
|
|
|
|
2008-07-13 22:04:41 +00:00
|
|
|
// Check to make sure that the other destination of this branch
|
|
|
|
// isn't BB itself. If so, this is an infinite loop that will
|
|
|
|
// keep getting unwound.
|
|
|
|
if (PBI->getSuccessor(PBIOp) == BB)
|
|
|
|
return false;
|
2008-07-13 21:53:26 +00:00
|
|
|
|
2008-07-13 22:04:41 +00:00
|
|
|
// Do not perform this transformation if it would require
|
|
|
|
// insertion of a large number of select instructions. For targets
|
|
|
|
// without predication/cmovs, this is a big pessimization.
|
|
|
|
BasicBlock *CommonDest = PBI->getSuccessor(PBIOp);
|
2008-07-13 21:53:26 +00:00
|
|
|
|
2008-07-13 22:04:41 +00:00
|
|
|
unsigned NumPhis = 0;
|
|
|
|
for (BasicBlock::iterator II = CommonDest->begin();
|
|
|
|
isa<PHINode>(II); ++II, ++NumPhis)
|
|
|
|
if (NumPhis > 2) // Disable this xform.
|
|
|
|
return false;
|
2008-07-13 21:53:26 +00:00
|
|
|
|
2008-07-13 22:04:41 +00:00
|
|
|
// Finally, if everything is ok, fold the branches to logical ops.
|
|
|
|
BasicBlock *OtherDest = BI->getSuccessor(BIOp ^ 1);
|
|
|
|
|
2010-01-05 01:26:52 +00:00
|
|
|
DEBUG(dbgs() << "FOLDING BRs:" << *PBI->getParent()
|
2009-08-23 04:37:46 +00:00
|
|
|
<< "AND: " << *BI->getParent());
|
2008-07-13 22:04:41 +00:00
|
|
|
|
2008-07-13 22:23:11 +00:00
|
|
|
|
|
|
|
// If OtherDest *is* BB, then BB is a basic block with a single conditional
|
|
|
|
// branch in it, where one edge (OtherDest) goes back to itself but the other
|
|
|
|
// exits. We don't *know* that the program avoids the infinite loop
|
|
|
|
// (even though that seems likely). If we do this xform naively, we'll end up
|
|
|
|
// recursively unpeeling the loop. Since we know that (after the xform is
|
|
|
|
// done) that the block *is* infinite if reached, we just make it an obviously
|
|
|
|
// infinite loop with no cond branch.
|
|
|
|
if (OtherDest == BB) {
|
|
|
|
// Insert it at the end of the function, because it's either code,
|
|
|
|
// or it won't matter if it's hot. :)
|
2009-08-13 21:58:54 +00:00
|
|
|
BasicBlock *InfLoopBlock = BasicBlock::Create(BB->getContext(),
|
|
|
|
"infloop", BB->getParent());
|
2008-07-13 22:23:11 +00:00
|
|
|
BranchInst::Create(InfLoopBlock, InfLoopBlock);
|
|
|
|
OtherDest = InfLoopBlock;
|
|
|
|
}
|
|
|
|
|
2010-01-05 01:26:52 +00:00
|
|
|
DEBUG(dbgs() << *PBI->getParent()->getParent());
|
2008-07-13 22:04:41 +00:00
|
|
|
|
|
|
|
// BI may have other predecessors. Because of this, we leave
|
|
|
|
// it alone, but modify PBI.
|
|
|
|
|
|
|
|
// Make sure we get to CommonDest on True&True directions.
|
|
|
|
Value *PBICond = PBI->getCondition();
|
|
|
|
if (PBIOp)
|
2009-08-12 16:23:25 +00:00
|
|
|
PBICond = BinaryOperator::CreateNot(PBICond,
|
2008-07-13 22:04:41 +00:00
|
|
|
PBICond->getName()+".not",
|
|
|
|
PBI);
|
|
|
|
Value *BICond = BI->getCondition();
|
|
|
|
if (BIOp)
|
2009-08-12 16:23:25 +00:00
|
|
|
BICond = BinaryOperator::CreateNot(BICond,
|
2008-07-13 22:04:41 +00:00
|
|
|
BICond->getName()+".not",
|
|
|
|
PBI);
|
|
|
|
// Merge the conditions.
|
|
|
|
Value *Cond = BinaryOperator::CreateOr(PBICond, BICond, "brmerge", PBI);
|
|
|
|
|
|
|
|
// Modify PBI to branch on the new condition to the new dests.
|
|
|
|
PBI->setCondition(Cond);
|
|
|
|
PBI->setSuccessor(0, CommonDest);
|
|
|
|
PBI->setSuccessor(1, OtherDest);
|
|
|
|
|
|
|
|
// OtherDest may have phi nodes. If so, add an entry from PBI's
|
|
|
|
// block that are identical to the entries for BI's block.
|
2010-12-14 07:09:42 +00:00
|
|
|
AddPredecessorToBlock(OtherDest, PBI->getParent(), BB);
|
2008-07-13 22:04:41 +00:00
|
|
|
|
|
|
|
// We know that the CommonDest already had an edge from PBI to
|
|
|
|
// it. If it has PHIs though, the PHIs may have different
|
|
|
|
// entries for BB and PBI's BB. If so, insert a select to make
|
|
|
|
// them agree.
|
2010-12-14 07:09:42 +00:00
|
|
|
PHINode *PN;
|
2008-07-13 22:04:41 +00:00
|
|
|
for (BasicBlock::iterator II = CommonDest->begin();
|
|
|
|
(PN = dyn_cast<PHINode>(II)); ++II) {
|
|
|
|
Value *BIV = PN->getIncomingValueForBlock(BB);
|
|
|
|
unsigned PBBIdx = PN->getBasicBlockIndex(PBI->getParent());
|
|
|
|
Value *PBIV = PN->getIncomingValue(PBBIdx);
|
|
|
|
if (BIV != PBIV) {
|
|
|
|
// Insert a select in PBI to pick the right value.
|
|
|
|
Value *NV = SelectInst::Create(PBICond, PBIV, BIV,
|
|
|
|
PBIV->getName()+".mux", PBI);
|
|
|
|
PN->setIncomingValue(PBBIdx, NV);
|
2008-07-13 21:53:26 +00:00
|
|
|
}
|
|
|
|
}
|
2008-07-13 22:04:41 +00:00
|
|
|
|
2010-01-05 01:26:52 +00:00
|
|
|
DEBUG(dbgs() << "INTO: " << *PBI->getParent());
|
|
|
|
DEBUG(dbgs() << *PBI->getParent()->getParent());
|
2008-07-13 22:04:41 +00:00
|
|
|
|
|
|
|
// This basic block is probably dead. We know it has at least
|
|
|
|
// one fewer predecessor.
|
|
|
|
return true;
|
2008-07-13 21:53:26 +00:00
|
|
|
}
|
|
|
|
|
2011-01-11 12:52:11 +00:00
|
|
|
// SimplifyTerminatorOnSelect - Simplifies a terminator by replacing it with a
|
|
|
|
// branch to TrueBB if Cond is true or to FalseBB if Cond is false.
|
|
|
|
// Takes care of updating the successors and removing the old terminator.
|
|
|
|
// Also makes sure not to introduce new successors by assuming that edges to
|
|
|
|
// non-successor TrueBBs and FalseBBs aren't reachable.
|
|
|
|
static bool SimplifyTerminatorOnSelect(TerminatorInst *OldTerm, Value *Cond,
|
|
|
|
BasicBlock *TrueBB, BasicBlock *FalseBB){
|
2010-12-05 18:29:03 +00:00
|
|
|
// Remove any superfluous successor edges from the CFG.
|
|
|
|
// First, figure out which successors to preserve.
|
|
|
|
// If TrueBB and FalseBB are equal, only try to preserve one copy of that
|
|
|
|
// successor.
|
|
|
|
BasicBlock *KeepEdge1 = TrueBB;
|
|
|
|
BasicBlock *KeepEdge2 = TrueBB != FalseBB ? FalseBB : 0;
|
|
|
|
|
|
|
|
// Then remove the rest.
|
2011-01-11 12:52:11 +00:00
|
|
|
for (unsigned I = 0, E = OldTerm->getNumSuccessors(); I != E; ++I) {
|
|
|
|
BasicBlock *Succ = OldTerm->getSuccessor(I);
|
2010-12-05 18:29:03 +00:00
|
|
|
// Make sure only to keep exactly one copy of each edge.
|
|
|
|
if (Succ == KeepEdge1)
|
|
|
|
KeepEdge1 = 0;
|
|
|
|
else if (Succ == KeepEdge2)
|
|
|
|
KeepEdge2 = 0;
|
|
|
|
else
|
2011-01-11 12:52:11 +00:00
|
|
|
Succ->removePredecessor(OldTerm->getParent());
|
2010-12-05 18:29:03 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// Insert an appropriate new terminator.
|
|
|
|
if ((KeepEdge1 == 0) && (KeepEdge2 == 0)) {
|
|
|
|
if (TrueBB == FalseBB)
|
|
|
|
// We were only looking for one successor, and it was present.
|
|
|
|
// Create an unconditional branch to it.
|
2011-01-11 12:52:11 +00:00
|
|
|
BranchInst::Create(TrueBB, OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
else
|
|
|
|
// We found both of the successors we were looking for.
|
|
|
|
// Create a conditional branch sharing the condition of the select.
|
2011-01-11 12:52:11 +00:00
|
|
|
BranchInst::Create(TrueBB, FalseBB, Cond, OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
} else if (KeepEdge1 && (KeepEdge2 || TrueBB == FalseBB)) {
|
|
|
|
// Neither of the selected blocks were successors, so this
|
2011-01-11 12:52:11 +00:00
|
|
|
// terminator must be unreachable.
|
|
|
|
new UnreachableInst(OldTerm->getContext(), OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
} else {
|
|
|
|
// One of the selected values was a successor, but the other wasn't.
|
|
|
|
// Insert an unconditional branch to the one that was found;
|
|
|
|
// the edge to the one that wasn't must be unreachable.
|
|
|
|
if (KeepEdge1 == 0)
|
|
|
|
// Only TrueBB was found.
|
2011-01-11 12:52:11 +00:00
|
|
|
BranchInst::Create(TrueBB, OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
else
|
|
|
|
// Only FalseBB was found.
|
2011-01-11 12:52:11 +00:00
|
|
|
BranchInst::Create(FalseBB, OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
}
|
|
|
|
|
2011-01-11 12:52:11 +00:00
|
|
|
EraseTerminatorInstAndDCECond(OldTerm);
|
2010-12-05 18:29:03 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2011-02-28 09:44:07 +00:00
|
|
|
// SimplifySwitchOnSelect - Replaces
|
|
|
|
// (switch (select cond, X, Y)) on constant X, Y
|
|
|
|
// with a branch - conditional if X and Y lead to distinct BBs,
|
|
|
|
// unconditional otherwise.
|
|
|
|
static bool SimplifySwitchOnSelect(SwitchInst *SI, SelectInst *Select) {
|
|
|
|
// Check for constant integer values in the select.
|
|
|
|
ConstantInt *TrueVal = dyn_cast<ConstantInt>(Select->getTrueValue());
|
|
|
|
ConstantInt *FalseVal = dyn_cast<ConstantInt>(Select->getFalseValue());
|
|
|
|
if (!TrueVal || !FalseVal)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Find the relevant condition and destinations.
|
|
|
|
Value *Condition = Select->getCondition();
|
|
|
|
BasicBlock *TrueBB = SI->getSuccessor(SI->findCaseValue(TrueVal));
|
|
|
|
BasicBlock *FalseBB = SI->getSuccessor(SI->findCaseValue(FalseVal));
|
|
|
|
|
|
|
|
// Perform the actual simplification.
|
|
|
|
return SimplifyTerminatorOnSelect(SI, Condition, TrueBB, FalseBB);
|
|
|
|
}
|
|
|
|
|
2011-01-11 12:52:11 +00:00
|
|
|
// SimplifyIndirectBrOnSelect - Replaces
|
|
|
|
// (indirectbr (select cond, blockaddress(@fn, BlockA),
|
|
|
|
// blockaddress(@fn, BlockB)))
|
|
|
|
// with
|
|
|
|
// (br cond, BlockA, BlockB).
|
|
|
|
static bool SimplifyIndirectBrOnSelect(IndirectBrInst *IBI, SelectInst *SI) {
|
|
|
|
// Check that both operands of the select are block addresses.
|
|
|
|
BlockAddress *TBA = dyn_cast<BlockAddress>(SI->getTrueValue());
|
|
|
|
BlockAddress *FBA = dyn_cast<BlockAddress>(SI->getFalseValue());
|
|
|
|
if (!TBA || !FBA)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Extract the actual blocks.
|
|
|
|
BasicBlock *TrueBB = TBA->getBasicBlock();
|
|
|
|
BasicBlock *FalseBB = FBA->getBasicBlock();
|
|
|
|
|
|
|
|
// Perform the actual simplification.
|
|
|
|
return SimplifyTerminatorOnSelect(IBI, SI->getCondition(), TrueBB, FalseBB);
|
|
|
|
}
|
|
|
|
|
fix a fairly serious oversight with switch formation from
or'd conditions. Previously we'd compile something like this:
int crud (unsigned char c) {
return c == 62 || c == 34 || c == 92;
}
into:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
]
lor.rhs: ; preds = %entry
%cmp8 = icmp eq i8 %c, 92
br label %lor.end
lor.end: ; preds = %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ %cmp8, %lor.rhs ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which failed to merge the compare-with-92 into the switch. With this patch
we simplify this all the way to:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
i8 92, label %lor.end
]
lor.rhs: ; preds = %entry
br label %lor.end
lor.end: ; preds = %entry, %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ false, %lor.rhs ], [ true, %entry ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which is much better for codegen's switch lowering stuff. This kicks in 33 times
on 176.gcc (for example) cutting 103 instructions off the generated code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121671 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 03:18:54 +00:00
|
|
|
/// TryToSimplifyUncondBranchWithICmpInIt - This is called when we find an icmp
|
|
|
|
/// instruction (a seteq/setne with a constant) as the only instruction in a
|
|
|
|
/// block that ends with an uncond branch. We are looking for a very specific
|
|
|
|
/// pattern that occurs when "A == 1 || A == 2 || A == 3" gets simplified. In
|
|
|
|
/// this case, we merge the first two "or's of icmp" into a switch, but then the
|
|
|
|
/// default value goes to an uncond block with a seteq in it, we get something
|
|
|
|
/// like:
|
|
|
|
///
|
|
|
|
/// switch i8 %A, label %DEFAULT [ i8 1, label %end i8 2, label %end ]
|
|
|
|
/// DEFAULT:
|
|
|
|
/// %tmp = icmp eq i8 %A, 92
|
|
|
|
/// br label %end
|
|
|
|
/// end:
|
|
|
|
/// ... = phi i1 [ true, %entry ], [ %tmp, %DEFAULT ], [ true, %entry ]
|
|
|
|
///
|
|
|
|
/// 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.
|
2010-12-14 06:17:25 +00:00
|
|
|
static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI,
|
|
|
|
const TargetData *TD) {
|
fix a fairly serious oversight with switch formation from
or'd conditions. Previously we'd compile something like this:
int crud (unsigned char c) {
return c == 62 || c == 34 || c == 92;
}
into:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
]
lor.rhs: ; preds = %entry
%cmp8 = icmp eq i8 %c, 92
br label %lor.end
lor.end: ; preds = %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ %cmp8, %lor.rhs ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which failed to merge the compare-with-92 into the switch. With this patch
we simplify this all the way to:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
i8 92, label %lor.end
]
lor.rhs: ; preds = %entry
br label %lor.end
lor.end: ; preds = %entry, %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ false, %lor.rhs ], [ true, %entry ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which is much better for codegen's switch lowering stuff. This kicks in 33 times
on 176.gcc (for example) cutting 103 instructions off the generated code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121671 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 03:18:54 +00:00
|
|
|
BasicBlock *BB = ICI->getParent();
|
|
|
|
// If the block has any PHIs in it or the icmp has multiple uses, it is too
|
|
|
|
// complex.
|
|
|
|
if (isa<PHINode>(BB->begin()) || !ICI->hasOneUse()) return false;
|
|
|
|
|
|
|
|
Value *V = ICI->getOperand(0);
|
|
|
|
ConstantInt *Cst = cast<ConstantInt>(ICI->getOperand(1));
|
|
|
|
|
|
|
|
// The pattern we're looking for is where our only predecessor is a switch on
|
|
|
|
// 'V' and this block is the default case for the switch. In this case we can
|
|
|
|
// fold the compared value into the switch to simplify things.
|
|
|
|
BasicBlock *Pred = BB->getSinglePredecessor();
|
|
|
|
if (Pred == 0 || !isa<SwitchInst>(Pred->getTerminator())) return false;
|
|
|
|
|
|
|
|
SwitchInst *SI = cast<SwitchInst>(Pred->getTerminator());
|
|
|
|
if (SI->getCondition() != V)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// If BB is reachable on a non-default case, then we simply know the value of
|
|
|
|
// V in this block. Substitute it and constant fold the icmp instruction
|
|
|
|
// away.
|
|
|
|
if (SI->getDefaultDest() != BB) {
|
|
|
|
ConstantInt *VVal = SI->findCaseDest(BB);
|
|
|
|
assert(VVal && "Should have a unique destination value");
|
|
|
|
ICI->setOperand(0, VVal);
|
|
|
|
|
2010-12-14 06:17:25 +00:00
|
|
|
if (Value *V = SimplifyInstruction(ICI, TD)) {
|
|
|
|
ICI->replaceAllUsesWith(V);
|
fix a fairly serious oversight with switch formation from
or'd conditions. Previously we'd compile something like this:
int crud (unsigned char c) {
return c == 62 || c == 34 || c == 92;
}
into:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
]
lor.rhs: ; preds = %entry
%cmp8 = icmp eq i8 %c, 92
br label %lor.end
lor.end: ; preds = %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ %cmp8, %lor.rhs ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which failed to merge the compare-with-92 into the switch. With this patch
we simplify this all the way to:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
i8 92, label %lor.end
]
lor.rhs: ; preds = %entry
br label %lor.end
lor.end: ; preds = %entry, %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ false, %lor.rhs ], [ true, %entry ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which is much better for codegen's switch lowering stuff. This kicks in 33 times
on 176.gcc (for example) cutting 103 instructions off the generated code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121671 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 03:18:54 +00:00
|
|
|
ICI->eraseFromParent();
|
|
|
|
}
|
|
|
|
// BB is now empty, so it is likely to simplify away.
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
}
|
|
|
|
|
2010-12-13 03:43:57 +00:00
|
|
|
// Ok, the block is reachable from the default dest. If the constant we're
|
|
|
|
// comparing exists in one of the other edges, then we can constant fold ICI
|
|
|
|
// and zap it.
|
|
|
|
if (SI->findCaseValue(Cst) != 0) {
|
|
|
|
Value *V;
|
|
|
|
if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
|
|
|
|
V = ConstantInt::getFalse(BB->getContext());
|
|
|
|
else
|
|
|
|
V = ConstantInt::getTrue(BB->getContext());
|
|
|
|
|
|
|
|
ICI->replaceAllUsesWith(V);
|
|
|
|
ICI->eraseFromParent();
|
|
|
|
// BB is now empty, so it is likely to simplify away.
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
}
|
|
|
|
|
fix a fairly serious oversight with switch formation from
or'd conditions. Previously we'd compile something like this:
int crud (unsigned char c) {
return c == 62 || c == 34 || c == 92;
}
into:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
]
lor.rhs: ; preds = %entry
%cmp8 = icmp eq i8 %c, 92
br label %lor.end
lor.end: ; preds = %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ %cmp8, %lor.rhs ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which failed to merge the compare-with-92 into the switch. With this patch
we simplify this all the way to:
switch i8 %c, label %lor.rhs [
i8 62, label %lor.end
i8 34, label %lor.end
i8 92, label %lor.end
]
lor.rhs: ; preds = %entry
br label %lor.end
lor.end: ; preds = %entry, %entry, %entry, %lor.rhs
%0 = phi i1 [ true, %entry ], [ false, %lor.rhs ], [ true, %entry ], [ true, %entry ]
%lor.ext = zext i1 %0 to i32
ret i32 %lor.ext
which is much better for codegen's switch lowering stuff. This kicks in 33 times
on 176.gcc (for example) cutting 103 instructions off the generated code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121671 91177308-0d34-0410-b5e6-96231b3b80d8
2010-12-13 03:18:54 +00:00
|
|
|
// The use of the icmp has to be in the 'end' block, by the only PHI node in
|
|
|
|
// the block.
|
|
|
|
BasicBlock *SuccBlock = BB->getTerminator()->getSuccessor(0);
|
|
|
|
PHINode *PHIUse = dyn_cast<PHINode>(ICI->use_back());
|
|
|
|
if (PHIUse == 0 || PHIUse != &SuccBlock->front() ||
|
|
|
|
isa<PHINode>(++BasicBlock::iterator(PHIUse)))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// If the icmp is a SETEQ, then the default dest gets false, the new edge gets
|
|
|
|
// true in the PHI.
|
|
|
|
Constant *DefaultCst = ConstantInt::getTrue(BB->getContext());
|
|
|
|
Constant *NewCst = ConstantInt::getFalse(BB->getContext());
|
|
|
|
|
|
|
|
if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
|
|
|
|
std::swap(DefaultCst, NewCst);
|
|
|
|
|
|
|
|
// Replace ICI (which is used by the PHI for the default value) with true or
|
|
|
|
// false depending on if it is EQ or NE.
|
|
|
|
ICI->replaceAllUsesWith(DefaultCst);
|
|
|
|
ICI->eraseFromParent();
|
|
|
|
|
|
|
|
// Okay, the switch goes to this block on a default value. Add an edge from
|
|
|
|
// the switch to the merge point on the compared value.
|
|
|
|
BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "switch.edge",
|
|
|
|
BB->getParent(), BB);
|
|
|
|
SI->addCase(Cst, NewBB);
|
|
|
|
|
|
|
|
// NewBB branches to the phi block, add the uncond branch and the phi entry.
|
|
|
|
BranchInst::Create(SuccBlock, NewBB);
|
|
|
|
PHIUse->addIncoming(NewCst, NewBB);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2010-12-13 05:03:41 +00:00
|
|
|
/// SimplifyBranchOnICmpChain - The specified branch is a conditional branch.
|
|
|
|
/// Check to see if it is branching on an or/and chain of icmp instructions, and
|
|
|
|
/// fold it into a switch instruction if so.
|
|
|
|
static bool SimplifyBranchOnICmpChain(BranchInst *BI, const TargetData *TD) {
|
|
|
|
Instruction *Cond = dyn_cast<Instruction>(BI->getCondition());
|
|
|
|
if (Cond == 0) return false;
|
|
|
|
|
|
|
|
|
|
|
|
// Change br (X == 0 | X == 1), T, F into a switch instruction.
|
|
|
|
// If this is a bunch of seteq's or'd together, or if it's a bunch of
|
|
|
|
// 'setne's and'ed together, collect them.
|
|
|
|
Value *CompVal = 0;
|
|
|
|
std::vector<ConstantInt*> Values;
|
|
|
|
bool TrueWhenEqual = true;
|
|
|
|
Value *ExtraCase = 0;
|
2011-02-07 22:37:28 +00:00
|
|
|
unsigned UsedICmps = 0;
|
2010-12-13 05:03:41 +00:00
|
|
|
|
|
|
|
if (Cond->getOpcode() == Instruction::Or) {
|
2011-02-07 22:37:28 +00:00
|
|
|
CompVal = GatherConstantCompares(Cond, Values, ExtraCase, TD, true,
|
|
|
|
UsedICmps);
|
2010-12-13 05:03:41 +00:00
|
|
|
} else if (Cond->getOpcode() == Instruction::And) {
|
2011-02-07 22:37:28 +00:00
|
|
|
CompVal = GatherConstantCompares(Cond, Values, ExtraCase, TD, false,
|
|
|
|
UsedICmps);
|
2010-12-13 05:03:41 +00:00
|
|
|
TrueWhenEqual = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we didn't have a multiply compared value, fail.
|
|
|
|
if (CompVal == 0) return false;
|
|
|
|
|
2011-02-07 22:37:28 +00:00
|
|
|
// Avoid turning single icmps into a switch.
|
|
|
|
if (UsedICmps <= 1)
|
|
|
|
return false;
|
|
|
|
|
2010-12-13 05:03:41 +00:00
|
|
|
// There might be duplicate constants in the list, which the switch
|
|
|
|
// instruction can't handle, remove them now.
|
|
|
|
array_pod_sort(Values.begin(), Values.end(), ConstantIntSortPredicate);
|
|
|
|
Values.erase(std::unique(Values.begin(), Values.end()), Values.end());
|
|
|
|
|
|
|
|
// If Extra was used, we require at least two switch values to do the
|
|
|
|
// transformation. A switch with one value is just an cond branch.
|
|
|
|
if (ExtraCase && Values.size() < 2) return false;
|
|
|
|
|
|
|
|
// Figure out which block is which destination.
|
|
|
|
BasicBlock *DefaultBB = BI->getSuccessor(1);
|
|
|
|
BasicBlock *EdgeBB = BI->getSuccessor(0);
|
|
|
|
if (!TrueWhenEqual) std::swap(DefaultBB, EdgeBB);
|
|
|
|
|
|
|
|
BasicBlock *BB = BI->getParent();
|
|
|
|
|
2010-12-14 06:17:25 +00:00
|
|
|
DEBUG(dbgs() << "Converting 'icmp' chain with " << Values.size()
|
2010-12-14 05:57:30 +00:00
|
|
|
<< " cases into SWITCH. BB is:\n" << *BB);
|
|
|
|
|
2010-12-13 05:03:41 +00:00
|
|
|
// If there are any extra values that couldn't be folded into the switch
|
|
|
|
// then we evaluate them with an explicit branch first. Split the block
|
|
|
|
// right before the condbr to handle it.
|
|
|
|
if (ExtraCase) {
|
|
|
|
BasicBlock *NewBB = BB->splitBasicBlock(BI, "switch.early.test");
|
|
|
|
// Remove the uncond branch added to the old block.
|
|
|
|
TerminatorInst *OldTI = BB->getTerminator();
|
|
|
|
|
2010-12-14 05:57:30 +00:00
|
|
|
if (TrueWhenEqual)
|
|
|
|
BranchInst::Create(EdgeBB, NewBB, ExtraCase, OldTI);
|
|
|
|
else
|
|
|
|
BranchInst::Create(NewBB, EdgeBB, ExtraCase, OldTI);
|
|
|
|
|
2010-12-13 05:03:41 +00:00
|
|
|
OldTI->eraseFromParent();
|
2010-12-13 05:34:18 +00:00
|
|
|
|
|
|
|
// If there are PHI nodes in EdgeBB, then we need to add a new entry to them
|
|
|
|
// for the edge we just added.
|
2010-12-14 07:09:42 +00:00
|
|
|
AddPredecessorToBlock(EdgeBB, BB, NewBB);
|
2010-12-14 06:17:25 +00:00
|
|
|
|
|
|
|
DEBUG(dbgs() << " ** 'icmp' chain unhandled condition: " << *ExtraCase
|
|
|
|
<< "\nEXTRABB = " << *BB);
|
2010-12-13 05:03:41 +00:00
|
|
|
BB = NewBB;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Convert pointer to int before we switch.
|
|
|
|
if (CompVal->getType()->isPointerTy()) {
|
|
|
|
assert(TD && "Cannot switch on pointer without TargetData");
|
|
|
|
CompVal = new PtrToIntInst(CompVal,
|
|
|
|
TD->getIntPtrType(CompVal->getContext()),
|
|
|
|
"magicptr", BI);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Create the new switch instruction now.
|
2010-12-13 06:25:44 +00:00
|
|
|
SwitchInst *New = SwitchInst::Create(CompVal, DefaultBB, Values.size(), BI);
|
2010-12-13 05:03:41 +00:00
|
|
|
|
|
|
|
// Add all of the 'cases' to the switch instruction.
|
|
|
|
for (unsigned i = 0, e = Values.size(); i != e; ++i)
|
|
|
|
New->addCase(Values[i], EdgeBB);
|
|
|
|
|
|
|
|
// We added edges from PI to the EdgeBB. As such, if there were any
|
|
|
|
// PHI nodes in EdgeBB, they need entries to be added corresponding to
|
|
|
|
// the number of edges added.
|
|
|
|
for (BasicBlock::iterator BBI = EdgeBB->begin();
|
|
|
|
isa<PHINode>(BBI); ++BBI) {
|
|
|
|
PHINode *PN = cast<PHINode>(BBI);
|
|
|
|
Value *InVal = PN->getIncomingValueForBlock(BB);
|
|
|
|
for (unsigned i = 0, e = Values.size()-1; i != e; ++i)
|
|
|
|
PN->addIncoming(InVal, BB);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Erase the old branch instruction.
|
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
2010-12-14 05:57:30 +00:00
|
|
|
|
2010-12-14 06:17:25 +00:00
|
|
|
DEBUG(dbgs() << " ** 'icmp' chain result is:\n" << *BB << '\n');
|
2010-12-13 05:03:41 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyCFGOpt::SimplifyReturn(ReturnInst *RI) {
|
|
|
|
BasicBlock *BB = RI->getParent();
|
|
|
|
if (!BB->getFirstNonPHIOrDbg()->isTerminator()) return false;
|
|
|
|
|
|
|
|
// Find predecessors that end with branches.
|
|
|
|
SmallVector<BasicBlock*, 8> UncondBranchPreds;
|
|
|
|
SmallVector<BranchInst*, 8> CondBranchPreds;
|
|
|
|
for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
|
|
|
|
BasicBlock *P = *PI;
|
|
|
|
TerminatorInst *PTI = P->getTerminator();
|
|
|
|
if (BranchInst *BI = dyn_cast<BranchInst>(PTI)) {
|
|
|
|
if (BI->isUnconditional())
|
|
|
|
UncondBranchPreds.push_back(P);
|
|
|
|
else
|
|
|
|
CondBranchPreds.push_back(BI);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we found some, do the transformation!
|
2011-01-29 04:46:23 +00:00
|
|
|
if (!UncondBranchPreds.empty() && DupRet) {
|
2010-12-13 06:25:44 +00:00
|
|
|
while (!UncondBranchPreds.empty()) {
|
|
|
|
BasicBlock *Pred = UncondBranchPreds.pop_back_val();
|
|
|
|
DEBUG(dbgs() << "FOLDING: " << *BB
|
|
|
|
<< "INTO UNCOND BRANCH PRED: " << *Pred);
|
2011-01-29 04:46:23 +00:00
|
|
|
(void)FoldReturnIntoUncondBranch(RI, BB, Pred);
|
2010-12-13 06:25:44 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// If we eliminated all predecessors of the block, delete the block now.
|
|
|
|
if (pred_begin(BB) == pred_end(BB))
|
|
|
|
// We know there are no successors, so just nuke the block.
|
|
|
|
BB->eraseFromParent();
|
|
|
|
|
2002-05-21 20:50:24 +00:00
|
|
|
return true;
|
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
// Check out all of the conditional branches going to this return
|
|
|
|
// instruction. If any of them just select between returns, change the
|
|
|
|
// branch itself into a select/return pair.
|
|
|
|
while (!CondBranchPreds.empty()) {
|
|
|
|
BranchInst *BI = CondBranchPreds.pop_back_val();
|
|
|
|
|
|
|
|
// Check to see if the non-BB successor is also a return block.
|
|
|
|
if (isa<ReturnInst>(BI->getSuccessor(0)->getTerminator()) &&
|
|
|
|
isa<ReturnInst>(BI->getSuccessor(1)->getTerminator()) &&
|
|
|
|
SimplifyCondBranchToTwoReturns(BI))
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
2002-05-21 20:50:24 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyCFGOpt::SimplifyUnwind(UnwindInst *UI) {
|
|
|
|
// Check to see if the first instruction in this block is just an unwind.
|
|
|
|
// If so, replace any invoke instructions which use this as an exception
|
|
|
|
// destination with call instructions.
|
|
|
|
BasicBlock *BB = UI->getParent();
|
|
|
|
if (!BB->getFirstNonPHIOrDbg()->isTerminator()) return false;
|
2009-10-30 22:39:04 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool Changed = false;
|
|
|
|
SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
|
|
|
|
while (!Preds.empty()) {
|
|
|
|
BasicBlock *Pred = Preds.back();
|
|
|
|
InvokeInst *II = dyn_cast<InvokeInst>(Pred->getTerminator());
|
|
|
|
if (II && II->getUnwindDest() == BB) {
|
|
|
|
// Insert a new branch instruction before the invoke, because this
|
|
|
|
// is now a fall through.
|
|
|
|
BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
|
|
|
|
Pred->getInstList().remove(II); // Take out of symbol table
|
|
|
|
|
|
|
|
// Insert the call now.
|
|
|
|
SmallVector<Value*,8> Args(II->op_begin(), II->op_end()-3);
|
|
|
|
CallInst *CI = CallInst::Create(II->getCalledValue(),
|
|
|
|
Args.begin(), Args.end(),
|
|
|
|
II->getName(), BI);
|
|
|
|
CI->setCallingConv(II->getCallingConv());
|
|
|
|
CI->setAttributes(II->getAttributes());
|
|
|
|
// If the invoke produced a value, the Call now does instead.
|
|
|
|
II->replaceAllUsesWith(CI);
|
|
|
|
delete II;
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
Preds.pop_back();
|
|
|
|
}
|
|
|
|
|
|
|
|
// If this block is now dead (and isn't the entry block), remove it.
|
|
|
|
if (pred_begin(BB) == pred_end(BB) &&
|
|
|
|
BB != &BB->getParent()->getEntryBlock()) {
|
|
|
|
// We know there are no successors, so just nuke the block.
|
|
|
|
BB->eraseFromParent();
|
2010-12-13 05:10:48 +00:00
|
|
|
return true;
|
2010-12-13 06:25:44 +00:00
|
|
|
}
|
2010-12-13 05:10:48 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
return Changed;
|
|
|
|
}
|
2008-03-11 21:53:06 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyCFGOpt::SimplifyUnreachable(UnreachableInst *UI) {
|
|
|
|
BasicBlock *BB = UI->getParent();
|
|
|
|
|
|
|
|
bool Changed = false;
|
|
|
|
|
|
|
|
// If there are any instructions immediately before the unreachable that can
|
|
|
|
// be removed, do so.
|
|
|
|
while (UI != BB->begin()) {
|
|
|
|
BasicBlock::iterator BBI = UI;
|
|
|
|
--BBI;
|
|
|
|
// Do not delete instructions that can have side effects, like calls
|
|
|
|
// (which may never return) and volatile loads and stores.
|
|
|
|
if (isa<CallInst>(BBI) && !isa<DbgInfoIntrinsic>(BBI)) break;
|
|
|
|
|
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(BBI))
|
|
|
|
if (SI->isVolatile())
|
|
|
|
break;
|
|
|
|
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(BBI))
|
|
|
|
if (LI->isVolatile())
|
|
|
|
break;
|
|
|
|
|
|
|
|
// Delete this instruction
|
2010-12-14 06:17:25 +00:00
|
|
|
BBI->eraseFromParent();
|
2010-12-13 06:25:44 +00:00
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If the unreachable instruction is the first in the block, take a gander
|
|
|
|
// at all of the predecessors of this instruction, and simplify them.
|
|
|
|
if (&BB->front() != UI) return Changed;
|
|
|
|
|
|
|
|
SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
|
|
|
|
for (unsigned i = 0, e = Preds.size(); i != e; ++i) {
|
|
|
|
TerminatorInst *TI = Preds[i]->getTerminator();
|
|
|
|
|
|
|
|
if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
|
|
|
|
if (BI->isUnconditional()) {
|
|
|
|
if (BI->getSuccessor(0) == BB) {
|
|
|
|
new UnreachableInst(TI->getContext(), TI);
|
|
|
|
TI->eraseFromParent();
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
if (BI->getSuccessor(0) == BB) {
|
|
|
|
BranchInst::Create(BI->getSuccessor(1), BI);
|
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
|
|
|
} else if (BI->getSuccessor(1) == BB) {
|
|
|
|
BranchInst::Create(BI->getSuccessor(0), BI);
|
|
|
|
EraseTerminatorInstAndDCECond(BI);
|
|
|
|
Changed = true;
|
2008-02-20 11:26:25 +00:00
|
|
|
}
|
2004-02-16 06:35:48 +00:00
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
} else if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
|
|
|
|
for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
|
|
|
|
if (SI->getSuccessor(i) == BB) {
|
|
|
|
BB->removePredecessor(SI->getParent());
|
|
|
|
SI->removeCase(i);
|
|
|
|
--i; --e;
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
// If the default value is unreachable, figure out the most popular
|
|
|
|
// destination and make it the default.
|
|
|
|
if (SI->getSuccessor(0) == BB) {
|
|
|
|
std::map<BasicBlock*, unsigned> Popularity;
|
|
|
|
for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
|
|
|
|
Popularity[SI->getSuccessor(i)]++;
|
|
|
|
|
|
|
|
// Find the most popular block.
|
|
|
|
unsigned MaxPop = 0;
|
|
|
|
BasicBlock *MaxBlock = 0;
|
|
|
|
for (std::map<BasicBlock*, unsigned>::iterator
|
|
|
|
I = Popularity.begin(), E = Popularity.end(); I != E; ++I) {
|
|
|
|
if (I->second > MaxPop) {
|
|
|
|
MaxPop = I->second;
|
|
|
|
MaxBlock = I->first;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (MaxBlock) {
|
|
|
|
// Make this the new default, allowing us to delete any explicit
|
|
|
|
// edges to it.
|
|
|
|
SI->setSuccessor(0, MaxBlock);
|
|
|
|
Changed = true;
|
2008-04-28 00:19:07 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
// If MaxBlock has phinodes in it, remove MaxPop-1 entries from
|
|
|
|
// it.
|
|
|
|
if (isa<PHINode>(MaxBlock->begin()))
|
|
|
|
for (unsigned i = 0; i != MaxPop-1; ++i)
|
|
|
|
MaxBlock->removePredecessor(SI->getParent());
|
|
|
|
|
|
|
|
for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
|
|
|
|
if (SI->getSuccessor(i) == MaxBlock) {
|
|
|
|
SI->removeCase(i);
|
|
|
|
--i; --e;
|
|
|
|
}
|
2004-02-16 06:35:48 +00:00
|
|
|
}
|
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
} else if (InvokeInst *II = dyn_cast<InvokeInst>(TI)) {
|
|
|
|
if (II->getUnwindDest() == BB) {
|
|
|
|
// Convert the invoke to a call instruction. This would be a good
|
|
|
|
// place to note that the call does not throw though.
|
2010-12-13 01:47:07 +00:00
|
|
|
BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
|
2010-12-13 06:25:44 +00:00
|
|
|
II->removeFromParent(); // Take out of symbol table
|
|
|
|
|
|
|
|
// Insert the call now...
|
|
|
|
SmallVector<Value*, 8> Args(II->op_begin(), II->op_end()-3);
|
2010-12-13 01:47:07 +00:00
|
|
|
CallInst *CI = CallInst::Create(II->getCalledValue(),
|
|
|
|
Args.begin(), Args.end(),
|
|
|
|
II->getName(), BI);
|
|
|
|
CI->setCallingConv(II->getCallingConv());
|
|
|
|
CI->setAttributes(II->getAttributes());
|
2010-12-13 06:25:44 +00:00
|
|
|
// If the invoke produced a value, the call does now instead.
|
2010-12-13 01:47:07 +00:00
|
|
|
II->replaceAllUsesWith(CI);
|
|
|
|
delete II;
|
|
|
|
Changed = true;
|
|
|
|
}
|
2004-02-24 05:54:22 +00:00
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// If this block is now dead, remove it.
|
|
|
|
if (pred_begin(BB) == pred_end(BB) &&
|
|
|
|
BB != &BB->getParent()->getEntryBlock()) {
|
|
|
|
// We know there are no successors, so just nuke the block.
|
|
|
|
BB->eraseFromParent();
|
|
|
|
return true;
|
|
|
|
}
|
2004-02-24 16:09:21 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
return Changed;
|
|
|
|
}
|
2004-02-24 16:09:21 +00:00
|
|
|
|
2011-02-02 15:56:22 +00:00
|
|
|
/// TurnSwitchRangeIntoICmp - Turns a switch with that contains only a
|
|
|
|
/// integer range comparison into a sub, an icmp and a branch.
|
|
|
|
static bool TurnSwitchRangeIntoICmp(SwitchInst *SI) {
|
|
|
|
assert(SI->getNumCases() > 2 && "Degenerate switch?");
|
2011-02-03 22:51:41 +00:00
|
|
|
|
|
|
|
// Make sure all cases point to the same destination and gather the values.
|
|
|
|
SmallVector<ConstantInt *, 16> Cases;
|
|
|
|
Cases.push_back(SI->getCaseValue(1));
|
|
|
|
for (unsigned I = 2, E = SI->getNumCases(); I != E; ++I) {
|
|
|
|
if (SI->getSuccessor(I-1) != SI->getSuccessor(I))
|
2011-02-02 15:56:22 +00:00
|
|
|
return false;
|
2011-02-03 22:51:41 +00:00
|
|
|
Cases.push_back(SI->getCaseValue(I));
|
|
|
|
}
|
|
|
|
assert(Cases.size() == SI->getNumCases()-1 && "Not all cases gathered");
|
|
|
|
|
|
|
|
// Sort the case values, then check if they form a range we can transform.
|
|
|
|
array_pod_sort(Cases.begin(), Cases.end(), ConstantIntSortPredicate);
|
|
|
|
for (unsigned I = 1, E = Cases.size(); I != E; ++I) {
|
|
|
|
if (Cases[I-1]->getValue() != Cases[I]->getValue()+1)
|
|
|
|
return false;
|
|
|
|
}
|
2011-02-02 15:56:22 +00:00
|
|
|
|
2011-02-03 22:51:41 +00:00
|
|
|
Constant *Offset = ConstantExpr::getNeg(Cases.back());
|
2011-02-02 15:56:22 +00:00
|
|
|
Constant *NumCases = ConstantInt::get(Offset->getType(), SI->getNumCases()-1);
|
|
|
|
|
2011-02-07 22:37:28 +00:00
|
|
|
Value *Sub = SI->getCondition();
|
|
|
|
if (!Offset->isNullValue())
|
|
|
|
Sub = BinaryOperator::CreateAdd(Sub, Offset, Sub->getName()+".off", SI);
|
2011-02-02 15:56:22 +00:00
|
|
|
Value *Cmp = new ICmpInst(SI, ICmpInst::ICMP_ULT, Sub, NumCases, "switch");
|
|
|
|
BranchInst::Create(SI->getSuccessor(1), SI->getDefaultDest(), Cmp, SI);
|
|
|
|
|
|
|
|
// Prune obsolete incoming values off the successor's PHI nodes.
|
|
|
|
for (BasicBlock::iterator BBI = SI->getSuccessor(1)->begin();
|
|
|
|
isa<PHINode>(BBI); ++BBI) {
|
|
|
|
for (unsigned I = 0, E = SI->getNumCases()-2; I != E; ++I)
|
|
|
|
cast<PHINode>(BBI)->removeIncomingValue(SI->getParent());
|
|
|
|
}
|
|
|
|
SI->eraseFromParent();
|
|
|
|
|
|
|
|
return true;
|
|
|
|
}
|
2009-02-04 21:39:48 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyCFGOpt::SimplifySwitch(SwitchInst *SI) {
|
|
|
|
// If this switch is too complex to want to look at, ignore it.
|
|
|
|
if (!isValueEqualityComparison(SI))
|
|
|
|
return false;
|
2004-05-01 23:35:43 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
BasicBlock *BB = SI->getParent();
|
|
|
|
|
|
|
|
// If we only have one predecessor, and if it is a branch on this value,
|
|
|
|
// see if that predecessor totally determines the outcome of this switch.
|
|
|
|
if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
|
|
|
|
if (SimplifyEqualityComparisonWithOnlyPredecessor(SI, OnlyPred))
|
2010-12-13 06:36:51 +00:00
|
|
|
return SimplifyCFG(BB) | true;
|
2011-02-28 09:44:07 +00:00
|
|
|
|
|
|
|
Value *Cond = SI->getCondition();
|
|
|
|
if (SelectInst *Select = dyn_cast<SelectInst>(Cond))
|
|
|
|
if (SimplifySwitchOnSelect(SI, Select))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
// If the block only contains the switch, see if we can fold the block
|
|
|
|
// away into any preds.
|
|
|
|
BasicBlock::iterator BBI = BB->begin();
|
|
|
|
// Ignore dbg intrinsics.
|
|
|
|
while (isa<DbgInfoIntrinsic>(BBI))
|
|
|
|
++BBI;
|
|
|
|
if (SI == &*BBI)
|
|
|
|
if (FoldValueComparisonIntoPredecessors(SI))
|
2010-12-13 06:36:51 +00:00
|
|
|
return SimplifyCFG(BB) | true;
|
2011-02-02 15:56:22 +00:00
|
|
|
|
|
|
|
// Try to transform the switch into an icmp and a branch.
|
|
|
|
if (TurnSwitchRangeIntoICmp(SI))
|
|
|
|
return SimplifyCFG(BB) | true;
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool SimplifyCFGOpt::SimplifyIndirectBr(IndirectBrInst *IBI) {
|
|
|
|
BasicBlock *BB = IBI->getParent();
|
|
|
|
bool Changed = false;
|
|
|
|
|
|
|
|
// Eliminate redundant destinations.
|
|
|
|
SmallPtrSet<Value *, 8> Succs;
|
|
|
|
for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) {
|
|
|
|
BasicBlock *Dest = IBI->getDestination(i);
|
|
|
|
if (!Dest->hasAddressTaken() || !Succs.insert(Dest)) {
|
|
|
|
Dest->removePredecessor(BB);
|
|
|
|
IBI->removeDestination(i);
|
|
|
|
--i; --e;
|
2004-10-18 04:07:22 +00:00
|
|
|
Changed = true;
|
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
}
|
2004-10-18 04:07:22 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
if (IBI->getNumDestinations() == 0) {
|
|
|
|
// If the indirectbr has no successors, change it to unreachable.
|
|
|
|
new UnreachableInst(IBI->getContext(), IBI);
|
|
|
|
EraseTerminatorInstAndDCECond(IBI);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (IBI->getNumDestinations() == 1) {
|
|
|
|
// If the indirectbr has one successor, change it to a direct branch.
|
|
|
|
BranchInst::Create(IBI->getDestination(0), IBI);
|
|
|
|
EraseTerminatorInstAndDCECond(IBI);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (SelectInst *SI = dyn_cast<SelectInst>(IBI->getAddress())) {
|
|
|
|
if (SimplifyIndirectBrOnSelect(IBI, SI))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
}
|
|
|
|
return Changed;
|
|
|
|
}
|
2004-10-18 04:07:22 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
bool SimplifyCFGOpt::SimplifyUncondBranch(BranchInst *BI) {
|
|
|
|
BasicBlock *BB = BI->getParent();
|
|
|
|
|
|
|
|
// If the Terminator is the only non-phi instruction, simplify the block.
|
|
|
|
BasicBlock::iterator I = BB->getFirstNonPHIOrDbg();
|
|
|
|
if (I->isTerminator() && BB != &BB->getParent()->getEntryBlock() &&
|
|
|
|
TryToSimplifyUncondBranchFromEmptyBlock(BB))
|
|
|
|
return true;
|
|
|
|
|
|
|
|
// If the only instruction in the block is a seteq/setne comparison
|
|
|
|
// against a constant, try to simplify the block.
|
|
|
|
if (ICmpInst *ICI = dyn_cast<ICmpInst>(I))
|
|
|
|
if (ICI->isEquality() && isa<ConstantInt>(ICI->getOperand(1))) {
|
|
|
|
for (++I; isa<DbgInfoIntrinsic>(I); ++I)
|
|
|
|
;
|
2010-12-14 06:17:25 +00:00
|
|
|
if (I->isTerminator() && TryToSimplifyUncondBranchWithICmpInIt(ICI, TD))
|
2004-10-18 04:07:22 +00:00
|
|
|
return true;
|
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
2010-08-14 00:29:42 +00:00
|
|
|
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
bool SimplifyCFGOpt::SimplifyCondBranch(BranchInst *BI) {
|
|
|
|
BasicBlock *BB = BI->getParent();
|
|
|
|
|
|
|
|
// Conditional branch
|
|
|
|
if (isValueEqualityComparison(BI)) {
|
|
|
|
// If we only have one predecessor, and if it is a branch on this value,
|
|
|
|
// see if that predecessor totally determines the outcome of this
|
|
|
|
// switch.
|
|
|
|
if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
|
|
|
|
if (SimplifyEqualityComparisonWithOnlyPredecessor(BI, OnlyPred))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
|
|
|
|
// This block must be empty, except for the setcond inst, if it exists.
|
|
|
|
// Ignore dbg intrinsics.
|
|
|
|
BasicBlock::iterator I = BB->begin();
|
|
|
|
// Ignore dbg intrinsics.
|
|
|
|
while (isa<DbgInfoIntrinsic>(I))
|
|
|
|
++I;
|
|
|
|
if (&*I == BI) {
|
|
|
|
if (FoldValueComparisonIntoPredecessors(BI))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
} else if (&*I == cast<Instruction>(BI->getCondition())){
|
|
|
|
++I;
|
|
|
|
// Ignore dbg intrinsics.
|
|
|
|
while (isa<DbgInfoIntrinsic>(I))
|
|
|
|
++I;
|
|
|
|
if (&*I == BI && FoldValueComparisonIntoPredecessors(BI))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Try to turn "br (X == 0 | X == 1), T, F" into a switch instruction.
|
|
|
|
if (SimplifyBranchOnICmpChain(BI, TD))
|
|
|
|
return true;
|
|
|
|
|
|
|
|
// We have a conditional branch to two blocks that are only reachable
|
|
|
|
// from BI. We know that the condbr dominates the two blocks, so see if
|
|
|
|
// there is any identical code in the "then" and "else" blocks. If so, we
|
|
|
|
// can hoist it up to the branching block.
|
|
|
|
if (BI->getSuccessor(0)->getSinglePredecessor() != 0) {
|
|
|
|
if (BI->getSuccessor(1)->getSinglePredecessor() != 0) {
|
|
|
|
if (HoistThenElseCodeToIf(BI))
|
2010-12-05 18:29:03 +00:00
|
|
|
return SimplifyCFG(BB) | true;
|
2010-12-13 06:25:44 +00:00
|
|
|
} else {
|
|
|
|
// If Successor #1 has multiple preds, we may be able to conditionally
|
|
|
|
// execute Successor #0 if it branches to successor #1.
|
|
|
|
TerminatorInst *Succ0TI = BI->getSuccessor(0)->getTerminator();
|
|
|
|
if (Succ0TI->getNumSuccessors() == 1 &&
|
|
|
|
Succ0TI->getSuccessor(0) == BI->getSuccessor(1))
|
|
|
|
if (SpeculativelyExecuteBB(BI, BI->getSuccessor(0)))
|
|
|
|
return SimplifyCFG(BB) | true;
|
2010-08-14 00:29:42 +00:00
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
} else if (BI->getSuccessor(1)->getSinglePredecessor() != 0) {
|
|
|
|
// If Successor #0 has multiple preds, we may be able to conditionally
|
|
|
|
// execute Successor #1 if it branches to successor #0.
|
|
|
|
TerminatorInst *Succ1TI = BI->getSuccessor(1)->getTerminator();
|
|
|
|
if (Succ1TI->getNumSuccessors() == 1 &&
|
|
|
|
Succ1TI->getSuccessor(0) == BI->getSuccessor(0))
|
|
|
|
if (SpeculativelyExecuteBB(BI, BI->getSuccessor(1)))
|
|
|
|
return SimplifyCFG(BB) | true;
|
2004-02-16 06:35:48 +00:00
|
|
|
}
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
// If this is a branch on a phi node in the current block, thread control
|
|
|
|
// through this block if any PHI node entries are constants.
|
|
|
|
if (PHINode *PN = dyn_cast<PHINode>(BI->getCondition()))
|
|
|
|
if (PN->getParent() == BI->getParent())
|
2010-12-14 06:17:25 +00:00
|
|
|
if (FoldCondBranchOnPHI(BI, TD))
|
2010-12-13 06:25:44 +00:00
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
|
|
|
|
// If this basic block is ONLY a setcc and a branch, and if a predecessor
|
|
|
|
// branches to us and one of our successors, fold the setcc into the
|
|
|
|
// predecessor and use logical operations to pick the right destination.
|
|
|
|
if (FoldBranchToCommonDest(BI))
|
2010-12-13 23:49:28 +00:00
|
|
|
return SimplifyCFG(BB) | true;
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
// Scan predecessor blocks for conditional branches.
|
|
|
|
for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI)
|
|
|
|
if (BranchInst *PBI = dyn_cast<BranchInst>((*PI)->getTerminator()))
|
|
|
|
if (PBI != BI && PBI->isConditional())
|
|
|
|
if (SimplifyCondBranchToCondBranch(PBI, BI))
|
|
|
|
return SimplifyCFG(BB) | true;
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool SimplifyCFGOpt::run(BasicBlock *BB) {
|
|
|
|
bool Changed = false;
|
|
|
|
|
2010-12-14 06:17:25 +00:00
|
|
|
assert(BB && BB->getParent() && "Block not embedded in function!");
|
2010-12-13 06:25:44 +00:00
|
|
|
assert(BB->getTerminator() && "Degenerate basic block encountered!");
|
|
|
|
|
|
|
|
// Remove basic blocks that have no predecessors (except the entry block)...
|
|
|
|
// or that just have themself as a predecessor. These are unreachable.
|
2010-12-14 06:17:25 +00:00
|
|
|
if ((pred_begin(BB) == pred_end(BB) &&
|
|
|
|
BB != &BB->getParent()->getEntryBlock()) ||
|
2010-12-13 06:25:44 +00:00
|
|
|
BB->getSinglePredecessor() == BB) {
|
|
|
|
DEBUG(dbgs() << "Removing BB: \n" << *BB);
|
|
|
|
DeleteDeadBlock(BB);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Check to see if we can constant propagate this terminator instruction
|
|
|
|
// away...
|
|
|
|
Changed |= ConstantFoldTerminator(BB);
|
|
|
|
|
|
|
|
// Check for and eliminate duplicate PHI nodes in this block.
|
|
|
|
Changed |= EliminateDuplicatePHINodes(BB);
|
|
|
|
|
|
|
|
// Merge basic blocks into their predecessor if there is only one distinct
|
|
|
|
// pred, and if there is only one distinct successor of the predecessor, and
|
|
|
|
// if there are no PHI nodes.
|
|
|
|
//
|
|
|
|
if (MergeBlockIntoPredecessor(BB))
|
|
|
|
return true;
|
|
|
|
|
|
|
|
// If there is a trivial two-entry PHI node in this basic block, and we can
|
|
|
|
// eliminate it, do so now.
|
|
|
|
if (PHINode *PN = dyn_cast<PHINode>(BB->begin()))
|
|
|
|
if (PN->getNumIncomingValues() == 2)
|
2010-12-14 07:00:00 +00:00
|
|
|
Changed |= FoldTwoEntryPHINode(PN, TD);
|
2010-12-13 06:25:44 +00:00
|
|
|
|
|
|
|
if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator())) {
|
2010-12-13 06:36:51 +00:00
|
|
|
if (BI->isUnconditional()) {
|
|
|
|
if (SimplifyUncondBranch(BI)) return true;
|
|
|
|
} else {
|
2010-12-14 05:57:30 +00:00
|
|
|
if (SimplifyCondBranch(BI)) return true;
|
2010-12-13 06:36:51 +00:00
|
|
|
}
|
|
|
|
} else if (ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator())) {
|
|
|
|
if (SimplifyReturn(RI)) return true;
|
|
|
|
} else if (SwitchInst *SI = dyn_cast<SwitchInst>(BB->getTerminator())) {
|
|
|
|
if (SimplifySwitch(SI)) return true;
|
|
|
|
} else if (UnreachableInst *UI =
|
|
|
|
dyn_cast<UnreachableInst>(BB->getTerminator())) {
|
|
|
|
if (SimplifyUnreachable(UI)) return true;
|
|
|
|
} else if (UnwindInst *UI = dyn_cast<UnwindInst>(BB->getTerminator())) {
|
|
|
|
if (SimplifyUnwind(UI)) return true;
|
|
|
|
} else if (IndirectBrInst *IBI =
|
|
|
|
dyn_cast<IndirectBrInst>(BB->getTerminator())) {
|
|
|
|
if (SimplifyIndirectBr(IBI)) return true;
|
2010-12-13 06:25:44 +00:00
|
|
|
}
|
|
|
|
|
2003-08-17 19:41:53 +00:00
|
|
|
return Changed;
|
2002-05-21 20:50:24 +00:00
|
|
|
}
|
2010-02-05 22:03:18 +00:00
|
|
|
|
|
|
|
/// SimplifyCFG - This function is used to do simplification of a CFG. For
|
|
|
|
/// example, it adjusts branches to branches to eliminate the extra hop, it
|
|
|
|
/// eliminates unreachable basic blocks, and does other "peephole" optimization
|
|
|
|
/// of the CFG. It returns true if a modification was made.
|
|
|
|
///
|
|
|
|
bool llvm::SimplifyCFG(BasicBlock *BB, const TargetData *TD) {
|
|
|
|
return SimplifyCFGOpt(TD).run(BB);
|
|
|
|
}
|