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

[sanitizer-coverage] Mark the guard and 8-bit counter arrays as used 

In r311742 we marked the PCs array as used so it wouldn't be dead
stripped, but left the guard and 8-bit counters arrays alone since
these are referenced by the coverage instrumentation. This doesn't
quite work if we want the indices of the PCs array to match the other
arrays though, since elements can still end up being dead and
disappear.

Instead, we mark all three of these arrays as used so that they'll be
consistent with one another.

llvm-svn: 311959
This commit is contained in:
Justin Bogner 2017-08-29 00:11:05 +00:00
parent 24bc6a4c4f
commit f1a54a47b0
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
View File

@ -566,23 +566,30 @@ SanitizerCoverageModule::CreatePCArray(Function &F,
ConstantArray::get(ArrayType::get(IntptrPtrTy, N * 2), PCs));
PCArray->setConstant(true);
// We don't reference the PCs array in any of our runtime functions, so we
// need to prevent it from being dead stripped.
appendToUsed(*F.getParent(), {PCArray});
return PCArray;
}
void SanitizerCoverageModule::CreateFunctionLocalArrays(
Function &F, ArrayRef<BasicBlock *> AllBlocks) {
if (Options.TracePCGuard)
SmallVector<GlobalValue *, 3> LocalArrays;
if (Options.TracePCGuard) {
FunctionGuardArray = CreateFunctionLocalArrayInSection(
AllBlocks.size(), F, Int32Ty, SanCovGuardsSectionName);
if (Options.Inline8bitCounters)
LocalArrays.push_back(FunctionGuardArray);
}
if (Options.Inline8bitCounters) {
Function8bitCounterArray = CreateFunctionLocalArrayInSection(
AllBlocks.size(), F, Int8Ty, SanCovCountersSectionName);
if (Options.PCTable)
LocalArrays.push_back(Function8bitCounterArray);
}
if (Options.PCTable) {
FunctionPCsArray = CreatePCArray(F, AllBlocks);
LocalArrays.push_back(FunctionPCsArray);
}
// We don't reference these arrays directly in any of our runtime functions,
// so we need to prevent them from being dead stripped.
appendToUsed(*F.getParent(), LocalArrays);
}
bool SanitizerCoverageModule::InjectCoverage(Function &F,