RPCS3/llvm
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm.git synced 2025-02-12 13:48:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[sanitizer-coverage] call appendToUsed once per module, not once per function (which is too slow)

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@312855 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Kostya Serebryany 2017-09-09 05:30:13 +00:00
parent c2b919492b
commit eba4372133
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/Transforms/Instrumentation/SanitizerCoverage.cpp
View File

@ -242,6 +242,7 @@ private:
GlobalVariable *FunctionGuardArray; // for trace-pc-guard.
GlobalVariable *Function8bitCounterArray; // for inline-8bit-counters.
GlobalVariable *FunctionPCsArray; // for pc-table.
SmallVector<GlobalValue *, 20> GlobalsToAppendToUsed;
SanitizerCoverageOptions Options;
};
@ -400,6 +401,10 @@ bool SanitizerCoverageModule::runOnModule(Module &M) {
{IRB.CreatePointerCast(SecStartEnd.first, IntptrPtrTy),
IRB.CreatePointerCast(SecStartEnd.second, IntptrPtrTy)});
}
// We don't reference these arrays directly in any of our runtime functions,
// so we need to prevent them from being dead stripped.
if (TargetTriple.isOSBinFormatMachO())
appendToUsed(M, GlobalsToAppendToUsed);
return true;
}
@ -579,25 +584,20 @@ SanitizerCoverageModule::CreatePCArray(Function &F,
void SanitizerCoverageModule::CreateFunctionLocalArrays(
Function &F, ArrayRef<BasicBlock *> AllBlocks) {
SmallVector<GlobalValue *, 3> LocalArrays;
if (Options.TracePCGuard) {
FunctionGuardArray = CreateFunctionLocalArrayInSection(
AllBlocks.size(), F, Int32Ty, SanCovGuardsSectionName);
LocalArrays.push_back(FunctionGuardArray);
GlobalsToAppendToUsed.push_back(FunctionGuardArray);
}
if (Options.Inline8bitCounters) {
Function8bitCounterArray = CreateFunctionLocalArrayInSection(
AllBlocks.size(), F, Int8Ty, SanCovCountersSectionName);
LocalArrays.push_back(Function8bitCounterArray);
GlobalsToAppendToUsed.push_back(Function8bitCounterArray);
}
if (Options.PCTable) {
FunctionPCsArray = CreatePCArray(F, AllBlocks);
LocalArrays.push_back(FunctionPCsArray);
GlobalsToAppendToUsed.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,