summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
diff options
context:
space:
mode:
authorEaswaran Raman <eraman@google.com>2018-01-25 19:27:17 +0000
committerEaswaran Raman <eraman@google.com>2018-01-25 19:27:17 +0000
commitc73cec84c99e5a63dca961fef67998a677c53a3c (patch)
tree704fa86d10c8220cb72ecd5c867b540340ea70d8 /llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
parent7e9453d5837f70284244dca2143fa5d8e491f955 (diff)
downloadbcm5719-llvm-c73cec84c99e5a63dca961fef67998a677c53a3c.tar.gz
bcm5719-llvm-c73cec84c99e5a63dca961fef67998a677c53a3c.zip
Re-land "[ThinLTO] Add call edges' relative block frequency to per-module summary."
It was reverted after buildbot regressions. Original commit message: This allows relative block frequency of call edges to be passed to the thinlink stage where it will be used to compute synthetic entry counts of functions. llvm-svn: 323460
Diffstat (limited to 'llvm/lib/Analysis/ModuleSummaryAnalysis.cpp')
-rw-r--r--llvm/lib/Analysis/ModuleSummaryAnalysis.cpp21
1 files changed, 18 insertions, 3 deletions
diff --git a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
index cf2fe7776dd..9293f603479 100644
--- a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
+++ b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
@@ -273,9 +273,24 @@ computeFunctionSummary(ModuleSummaryIndex &Index, const Module &M,
// to record the call edge to the alias in that case. Eventually
// an alias summary will be created to associate the alias and
// aliasee.
- CallGraphEdges[Index.getOrInsertValueInfo(
- cast<GlobalValue>(CalledValue))]
- .updateHotness(Hotness);
+ auto &ValueInfo = CallGraphEdges[Index.getOrInsertValueInfo(
+ cast<GlobalValue>(CalledValue))];
+ ValueInfo.updateHotness(Hotness);
+ // Add the relative block frequency to CalleeInfo if there is no profile
+ // information.
+ if (BFI != nullptr && Hotness == CalleeInfo::HotnessType::Unknown) {
+ auto BBFreq = BFI->getBlockFreq(&BB).getFrequency();
+ // FIXME: This might need some scaling to prevent BBFreq values from
+ // being rounded down to 0.
+ auto EntryFreq = BFI->getEntryFreq();
+ // Block frequencies can be directly set for a block and so we need to
+ // handle the case of entry frequency being 0.
+ if (EntryFreq)
+ BBFreq /= EntryFreq;
+ else
+ BBFreq = 0;
+ ValueInfo.updateRelBlockFreq(BBFreq);
+ }
} else {
// Skip inline assembly calls.
if (CI && CI->isInlineAsm())
OpenPOWER on IntegriCloud