maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Move all the calls to AA.getTargetLibraryInfo() to using a TLI member variable. 

No functionality change.

llvm-svn: 164560
This commit is contained in:
Nick Lewycky 2012-09-24 22:07:09 +00:00
parent 14fd801f8f
commit 135ac9ac89
1 changed files with 11 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
View File

@ -45,6 +45,7 @@ namespace {
AliasAnalysis *AA; AliasAnalysis *AA;
MemoryDependenceAnalysis *MD; MemoryDependenceAnalysis *MD;
DominatorTree *DT; DominatorTree *DT;
const TargetLibraryInfo *TLI;
static char ID; // Pass identification, replacement for typeid static char ID; // Pass identification, replacement for typeid
DSE() : FunctionPass(ID), AA(0), MD(0), DT(0) { DSE() : FunctionPass(ID), AA(0), MD(0), DT(0) {
@ -55,6 +56,7 @@ namespace {
AA = &getAnalysis<AliasAnalysis>(); AA = &getAnalysis<AliasAnalysis>();
MD = &getAnalysis<MemoryDependenceAnalysis>(); MD = &getAnalysis<MemoryDependenceAnalysis>();
DT = &getAnalysis<DominatorTree>(); DT = &getAnalysis<DominatorTree>();
TLI = AA->getTargetLibraryInfo();
bool Changed = false; bool Changed = false;
for (Function::iterator I = F.begin(), E = F.end(); I != E; ++I) for (Function::iterator I = F.begin(), E = F.end(); I != E; ++I)
@ -455,7 +457,7 @@ bool DSE::runOnBasicBlock(BasicBlock &BB) {
Instruction *Inst = BBI++; Instruction *Inst = BBI++;
// Handle 'free' calls specially. // Handle 'free' calls specially.
if (CallInst *F = isFreeCall(Inst, AA->getTargetLibraryInfo())) { if (CallInst *F = isFreeCall(Inst, TLI)) {
MadeChange |= HandleFree(F); MadeChange |= HandleFree(F);
continue; continue;
} }
@ -484,7 +486,7 @@ bool DSE::runOnBasicBlock(BasicBlock &BB) {
// in case we need it. // in case we need it.
WeakVH NextInst(BBI); WeakVH NextInst(BBI);
DeleteDeadInstruction(SI, *MD, AA->getTargetLibraryInfo()); DeleteDeadInstruction(SI, *MD, TLI);
if (NextInst == 0) // Next instruction deleted. if (NextInst == 0) // Next instruction deleted.
BBI = BB.begin(); BBI = BB.begin();
@ -531,7 +533,7 @@ bool DSE::runOnBasicBlock(BasicBlock &BB) {
<< *DepWrite << "\n KILLER: " << *Inst << '\n'); << *DepWrite << "\n KILLER: " << *Inst << '\n');
// Delete the store and now-dead instructions that feed it. // Delete the store and now-dead instructions that feed it.
DeleteDeadInstruction(DepWrite, *MD, AA->getTargetLibraryInfo()); DeleteDeadInstruction(DepWrite, *MD, TLI);
++NumFastStores; ++NumFastStores;
MadeChange = true; MadeChange = true;
@ -641,7 +643,7 @@ bool DSE::HandleFree(CallInst *F) {
Instruction *Next = llvm::next(BasicBlock::iterator(Dependency)); Instruction *Next = llvm::next(BasicBlock::iterator(Dependency));
// DCE instructions only used to calculate that store // DCE instructions only used to calculate that store
DeleteDeadInstruction(Dependency, *MD, AA->getTargetLibraryInfo()); DeleteDeadInstruction(Dependency, *MD, TLI);
++NumFastStores; ++NumFastStores;
MadeChange = true; MadeChange = true;
@ -681,8 +683,7 @@ bool DSE::handleEndBlock(BasicBlock &BB) {
// Okay, so these are dead heap objects, but if the pointer never escapes // Okay, so these are dead heap objects, but if the pointer never escapes
// then it's leaked by this function anyways. // then it's leaked by this function anyways.
else if (isAllocLikeFn(I, AA->getTargetLibraryInfo()) && else if (isAllocLikeFn(I, TLI) && !PointerMayBeCaptured(I, true, true))
!PointerMayBeCaptured(I, true, true))
DeadStackObjects.insert(I); DeadStackObjects.insert(I);
} }
@ -726,8 +727,7 @@ bool DSE::handleEndBlock(BasicBlock &BB) {
dbgs() << '\n'); dbgs() << '\n');
// DCE instructions only used to calculate that store. // DCE instructions only used to calculate that store.
DeleteDeadInstruction(Dead, *MD, AA->getTargetLibraryInfo(), DeleteDeadInstruction(Dead, *MD, TLI, &DeadStackObjects);
&DeadStackObjects);
++NumFastStores; ++NumFastStores;
MadeChange = true; MadeChange = true;
continue; continue;
@ -735,10 +735,9 @@ bool DSE::handleEndBlock(BasicBlock &BB) {
} }
// Remove any dead non-memory-mutating instructions. // Remove any dead non-memory-mutating instructions.
if (isInstructionTriviallyDead(BBI, AA->getTargetLibraryInfo())) { if (isInstructionTriviallyDead(BBI, TLI)) {
Instruction *Inst = BBI++; Instruction *Inst = BBI++;
DeleteDeadInstruction(Inst, *MD, AA->getTargetLibraryInfo(), DeleteDeadInstruction(Inst, *MD, TLI, &DeadStackObjects);
&DeadStackObjects);
++NumFastOther; ++NumFastOther;
MadeChange = true; MadeChange = true;
continue; continue;
@ -754,7 +753,7 @@ bool DSE::handleEndBlock(BasicBlock &BB) {
if (CallSite CS = cast<Value>(BBI)) { if (CallSite CS = cast<Value>(BBI)) {
// Remove allocation function calls from the list of dead stack objects; // Remove allocation function calls from the list of dead stack objects;
// there can't be any references before the definition. // there can't be any references before the definition.
if (isAllocLikeFn(BBI, AA->getTargetLibraryInfo())) if (isAllocLikeFn(BBI, TLI))
DeadStackObjects.remove(BBI); DeadStackObjects.remove(BBI);
// If this call does not access memory, it can't be loading any of our // If this call does not access memory, it can't be loading any of our