diff options
author | Easwaran Raman <eraman@google.com> | 2017-12-22 01:33:52 +0000 |
---|---|---|
committer | Easwaran Raman <eraman@google.com> | 2017-12-22 01:33:52 +0000 |
commit | a17f220590edc34b6c59c385c59312b3e9da524c (patch) | |
tree | e87b601432930780096b04c0bb50c0028c42ece6 /llvm/lib/Analysis/ModuleSummaryAnalysis.cpp | |
parent | b4ba1aa486adee904002cdf5038133512b4ed37b (diff) | |
download | llvm-a17f220590edc34b6c59c385c59312b3e9da524c.zip llvm-a17f220590edc34b6c59c385c59312b3e9da524c.tar.gz llvm-a17f220590edc34b6c59c385c59312b3e9da524c.tar.bz2 |
Add hasProfileData() to check if a function has profile data. NFC.
Summary:
This replaces calls to getEntryCount().hasValue() with hasProfileData
that does the same thing. This refactoring is useful to do before adding
synthetic function entry counts but also a useful cleanup IMO even
otherwise. I have used hasProfileData instead of hasRealProfileData as
David had earlier suggested since I think profile implies "real" and I
use the phrase "synthetic entry count" and not "synthetic profile count"
but I am fine calling it hasRealProfileData if you prefer.
Reviewers: davidxl, silvas
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D41461
llvm-svn: 321331
Diffstat (limited to 'llvm/lib/Analysis/ModuleSummaryAnalysis.cpp')
-rw-r--r-- | llvm/lib/Analysis/ModuleSummaryAnalysis.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp index d54fb70..10badd8 100644 --- a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp +++ b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp @@ -454,7 +454,7 @@ ModuleSummaryIndex llvm::buildModuleSummaryIndex( std::unique_ptr<BlockFrequencyInfo> BFIPtr; if (GetBFICallback) BFI = GetBFICallback(F); - else if (F.getEntryCount().hasValue()) { + else if (F.hasProfileData()) { LoopInfo LI{DominatorTree(const_cast<Function &>(F))}; BranchProbabilityInfo BPI{F, LI}; BFIPtr = llvm::make_unique<BlockFrequencyInfo>(F, BPI, LI); |