replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Rename PointsToLocalMemory to PointsToLocalOrConstantMemory to make 

the code more self-documenting.

llvm-svn: 118171
This commit is contained in:
Duncan Sands 2010-11-03 14:45:05 +00:00
parent b2aa0ef310
commit 9d1fe4c40d
1 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
llvm/lib/Transforms/IPO/FunctionAttrs.cpp
View File

@ -65,7 +65,7 @@ namespace {
CallGraphSCCPass::getAnalysisUsage(AU); CallGraphSCCPass::getAnalysisUsage(AU);
} }
bool PointsToLocalMemory(Value *V); bool PointsToLocalOrConstantMemory(Value *V);
}; };
} }
@ -79,10 +79,10 @@ INITIALIZE_PASS_END(FunctionAttrs, "functionattrs",
Pass *llvm::createFunctionAttrsPass() { return new FunctionAttrs(); } Pass *llvm::createFunctionAttrsPass() { return new FunctionAttrs(); }
/// PointsToLocalMemory - Returns whether the given pointer value points to /// PointsToLocalOrConstantMemory - Returns whether the given pointer value
/// memory that is local to the function. Global constants are considered /// points to memory that is local to the function, with global constants being
/// local to all functions. /// considered local to all functions.
bool FunctionAttrs::PointsToLocalMemory(Value *V) { bool FunctionAttrs::PointsToLocalOrConstantMemory(Value *V) {
SmallVector<Value*, 16> Worklist; SmallVector<Value*, 16> Worklist;
unsigned MaxLookup = 8; unsigned MaxLookup = 8;
@ -179,7 +179,8 @@ bool FunctionAttrs::AddReadAttrs(const CallGraphSCC &SCC) {
for (CallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end(); for (CallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end();
CI != CE; ++CI) { CI != CE; ++CI) {
Value *Arg = *CI; Value *Arg = *CI;
if (Arg->getType()->isPointerTy() && !PointsToLocalMemory(Arg)) if (Arg->getType()->isPointerTy() &&
!PointsToLocalOrConstantMemory(Arg))
// Writes memory. Just give up. // Writes memory. Just give up.
return false; return false;
} }
@ -188,11 +189,13 @@ bool FunctionAttrs::AddReadAttrs(const CallGraphSCC &SCC) {
} }
} else if (LoadInst *LI = dyn_cast<LoadInst>(I)) { } else if (LoadInst *LI = dyn_cast<LoadInst>(I)) {
// Ignore non-volatile loads from local memory. // Ignore non-volatile loads from local memory.
if (!LI->isVolatile() && PointsToLocalMemory(LI->getPointerOperand())) if (!LI->isVolatile() &&
PointsToLocalOrConstantMemory(LI->getPointerOperand()))
continue; continue;
} else if (StoreInst *SI = dyn_cast<StoreInst>(I)) { } else if (StoreInst *SI = dyn_cast<StoreInst>(I)) {
// Ignore non-volatile stores to local memory. // Ignore non-volatile stores to local memory.
if (!SI->isVolatile() && PointsToLocalMemory(SI->getPointerOperand())) if (!SI->isVolatile() &&
PointsToLocalOrConstantMemory(SI->getPointerOperand()))
continue; continue;
} }