From 6891db977b6f6ec9dfd59b807082096b7f48f442 Mon Sep 17 00:00:00 2001 From: Xinliang David Li Date: Tue, 17 May 2016 20:19:03 +0000 Subject: [PATCH] Simple refactoring /NFC llvm-svn: 269829 --- .../Instrumentation/InstrProfiling.cpp | 23 ++++++++++++------- 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/lib/Transforms/Instrumentation/InstrProfiling.cpp b/lib/Transforms/Instrumentation/InstrProfiling.cpp index 930001fd684..60f77d47fb4 100644 --- a/lib/Transforms/Instrumentation/InstrProfiling.cpp +++ b/lib/Transforms/Instrumentation/InstrProfiling.cpp @@ -367,6 +367,20 @@ InstrProfiling::getOrCreateRegionCounters(InstrProfIncrementInst *Inc) { return CounterPtr; } +static bool needsRuntimeRegistrationOfSectionRange(const Module &M) { + // Don't do this for Darwin. compiler-rt uses linker magic. + if (Triple(M.getTargetTriple()).isOSDarwin()) + return false; + + // Use linker script magic to get data/cnts/name start/end. + if (Triple(M.getTargetTriple()).isOSLinux() || + Triple(M.getTargetTriple()).isOSFreeBSD() || + Triple(M.getTargetTriple()).isPS4CPU()) + return false; + + return true; +} + void InstrProfiling::emitNameData() { std::string UncompressedData; @@ -391,14 +405,7 @@ void InstrProfiling::emitNameData() { } void InstrProfiling::emitRegistration() { - // Don't do this for Darwin. compiler-rt uses linker magic. - if (Triple(M->getTargetTriple()).isOSDarwin()) - return; - - // Use linker script magic to get data/cnts/name start/end. - if (Triple(M->getTargetTriple()).isOSLinux() || - Triple(M->getTargetTriple()).isOSFreeBSD() || - Triple(M->getTargetTriple()).isPS4CPU()) + if (!needsRuntimeRegistrationOfSectionRange(*M)) return; // Construct the function.