diff options
author | Daniel Dunbar <daniel@zuster.org> | 2009-08-05 21:51:16 +0000 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2009-08-05 21:51:16 +0000 |
commit | c9008c5cc7113ea4c3a262e346c0dfcdbca12ae6 (patch) | |
tree | e8e8c7d27032dc84dd20ce8327a4cbf927104d1f /lib/Analysis | |
parent | b341577401dfb25e1e6c451eee6dd01df531365a (diff) | |
download | external_llvm-c9008c5cc7113ea4c3a262e346c0dfcdbca12ae6.zip external_llvm-c9008c5cc7113ea4c3a262e346c0dfcdbca12ae6.tar.gz external_llvm-c9008c5cc7113ea4c3a262e346c0dfcdbca12ae6.tar.bz2 |
Make block and function count available via ProfileInfo.
- Part of optimal static profiling patch sequence by Andreas Neustifter.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@78247 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis')
-rw-r--r-- | lib/Analysis/ProfileInfo.cpp | 7 | ||||
-rw-r--r-- | lib/Analysis/ProfileInfoLoaderPass.cpp | 18 |
2 files changed, 19 insertions, 6 deletions
diff --git a/lib/Analysis/ProfileInfo.cpp b/lib/Analysis/ProfileInfo.cpp index 26328d0..1b86ec8 100644 --- a/lib/Analysis/ProfileInfo.cpp +++ b/lib/Analysis/ProfileInfo.cpp @@ -27,6 +27,9 @@ char ProfileInfo::ID = 0; ProfileInfo::~ProfileInfo() {} unsigned ProfileInfo::getExecutionCount(const BasicBlock *BB) const { + if (BlockCounts.find(BB) != BlockCounts.end()) + return BlockCounts.find(BB)->second; + pred_const_iterator PI = pred_begin(BB), PE = pred_end(BB); // Are there zero predecessors of this block? @@ -76,7 +79,9 @@ unsigned ProfileInfo::getExecutionCount(const BasicBlock *BB) const { } unsigned ProfileInfo::getExecutionCount(const Function *F) const { - if (F->isDeclaration()) return -1; + if (FunctionCounts.find(F) != FunctionCounts.end()) + return FunctionCounts.find(F)->second; + return getExecutionCount(&F->getEntryBlock()); } diff --git a/lib/Analysis/ProfileInfoLoaderPass.cpp b/lib/Analysis/ProfileInfoLoaderPass.cpp index 2d9c8b9..323174b 100644 --- a/lib/Analysis/ProfileInfoLoaderPass.cpp +++ b/lib/Analysis/ProfileInfoLoaderPass.cpp @@ -72,21 +72,29 @@ bool LoaderPass::runOnModule(Module &M) { EdgeCounts.clear(); std::vector<unsigned> ECs = PIL.getRawEdgeCounts(); + std::vector<unsigned> BCs = PIL.getRawBlockCounts(); + std::vector<unsigned> FCs = PIL.getRawFunctionCounts(); // Instrument all of the edges... - unsigned i = 0; - for (Module::iterator F = M.begin(), E = M.end(); F != E; ++F) + unsigned ei = 0; + unsigned bi = 0; + unsigned fi = 0; + for (Module::iterator F = M.begin(), E = M.end(); F != E; ++F) { + if (F->isDeclaration()) continue; + if (fi<FCs.size()) FunctionCounts[F] = FCs[fi++]; for (Function::iterator BB = F->begin(), E = F->end(); BB != E; ++BB) { + if (bi<BCs.size()) BlockCounts[BB] = BCs[bi++]; // Okay, we have to add a counter of each outgoing edge. If the // outgoing edge is not critical don't split it, just insert the counter // in the source or destination of the edge. TerminatorInst *TI = BB->getTerminator(); for (unsigned s = 0, e = TI->getNumSuccessors(); s != e; ++s) { - if (i < ECs.size()) - EdgeCounts[std::make_pair(BB, TI->getSuccessor(s))]+= ECs[i++]; + if (ei < ECs.size()) + EdgeCounts[std::make_pair(BB, TI->getSuccessor(s))]+= ECs[ei++]; } } + } - if (i != ECs.size()) { + if (ei != ECs.size()) { cerr << "WARNING: profile information is inconsistent with " << "the current program!\n"; } |