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

[tsan] typo and style (thanks to Nick Lewycky) 

llvm-svn: 155986
This commit is contained in:
Kostya Serebryany 2012-05-02 13:12:19 +00:00
parent 274ba89d77
commit ae7188d9b9
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp
View File

@ -71,8 +71,8 @@ struct ThreadSanitizer : public FunctionPass {
private:
bool instrumentLoadOrStore(Instruction *I);
bool instrumentAtomic(Instruction *I);
void choseInstructionsToInstrument(SmallVectorImpl<Instruction*> &Local,
SmallVectorImpl<Instruction*> &All);
void chooseInstructionsToInstrument(SmallVectorImpl<Instruction*> &Local,
SmallVectorImpl<Instruction*> &All);
bool addrPointsToConstantData(Value *Addr);
int getMemoryAccessFuncIndex(Value *Addr);
@ -207,7 +207,7 @@ bool ThreadSanitizer::addrPointsToConstantData(Value *Addr) {
//
// 'Local' is a vector of insns within the same BB (no calls between).
// 'All' is a vector of insns that will be instrumented.
void ThreadSanitizer::choseInstructionsToInstrument(
void ThreadSanitizer::chooseInstructionsToInstrument(
SmallVectorImpl<Instruction*> &Local,
SmallVectorImpl<Instruction*> &All) {
SmallSet<Value*, 8> WriteTargets;
@ -238,13 +238,13 @@ void ThreadSanitizer::choseInstructionsToInstrument(
static bool isAtomic(Instruction *I) {
if (LoadInst *LI = dyn_cast<LoadInst>(I))
return LI->isAtomic() && LI->getSynchScope() == CrossThread;
else if (StoreInst *SI = dyn_cast<StoreInst>(I))
if (StoreInst *SI = dyn_cast<StoreInst>(I))
return SI->isAtomic() && SI->getSynchScope() == CrossThread;
else if (isa<AtomicRMWInst>(I))
if (isa<AtomicRMWInst>(I))
return true;
else if (isa<AtomicCmpXchgInst>(I))
if (isa<AtomicCmpXchgInst>(I))
return true;
else if (FenceInst *FI = dyn_cast<FenceInst>(I))
if (FenceInst *FI = dyn_cast<FenceInst>(I))
return FI->getSynchScope() == CrossThread;
return false;
}
@ -273,10 +273,10 @@ bool ThreadSanitizer::runOnFunction(Function &F) {
RetVec.push_back(BI);
else if (isa<CallInst>(BI) || isa<InvokeInst>(BI)) {
HasCalls = true;
choseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores);
chooseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores);
}
}
choseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores);
chooseInstructionsToInstrument(LocalLoadsAndStores, AllLoadsAndStores);
}
// We have collected all loads and stores.