diff options
| author | Dehao Chen <dehao@google.com> | 2016-09-19 16:06:37 +0000 |
|---|---|---|
| committer | Dehao Chen <dehao@google.com> | 2016-09-19 16:06:37 +0000 |
| commit | 38e3731c470b569f8c61d7d919c726f48fca6521 (patch) | |
| tree | ce4075430897f382f2cc89f25cf78d464e1cfc90 /llvm/lib/Transforms/IPO/SampleProfile.cpp | |
| parent | dd6f8cab08f9f383c12319e5388b22ed77c66ff1 (diff) | |
| download | bcm5719-llvm-38e3731c470b569f8c61d7d919c726f48fca6521.tar.gz bcm5719-llvm-38e3731c470b569f8c61d7d919c726f48fca6521.zip | |
Use call target count to derive the call instruction weight
Summary: The call target count profile is directly derived from LBR branch->target data. This is more reliable than instruction frequency profiles that could be moved across basic block boundaries. This patches uses call target count profile to annotate call instructions.
Reviewers: davidxl, dnovillo
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D24410
llvm-svn: 281911
Diffstat (limited to 'llvm/lib/Transforms/IPO/SampleProfile.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/SampleProfile.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/IPO/SampleProfile.cpp b/llvm/lib/Transforms/IPO/SampleProfile.cpp index 9c4a40c44ca..500686529ad 100644 --- a/llvm/lib/Transforms/IPO/SampleProfile.cpp +++ b/llvm/lib/Transforms/IPO/SampleProfile.cpp @@ -480,7 +480,9 @@ SampleProfileLoader::getInstWeight(const Instruction &Inst) const { uint32_t LineOffset = getOffset(Lineno, HeaderLineno); uint32_t Discriminator = DIL->getDiscriminator(); - ErrorOr<uint64_t> R = FS->findSamplesAt(LineOffset, Discriminator); + ErrorOr<uint64_t> R = IsCall + ? FS->findCallSamplesAt(LineOffset, Discriminator) + : FS->findSamplesAt(LineOffset, Discriminator); if (R) { bool FirstMark = CoverageTracker.markSamplesUsed(FS, LineOffset, Discriminator, R.get()); @@ -1272,10 +1274,10 @@ bool SampleProfileLoader::emitAnnotations(Function &F) { char SampleProfileLoaderLegacyPass::ID = 0; INITIALIZE_PASS_BEGIN(SampleProfileLoaderLegacyPass, "sample-profile", - "Sample Profile loader", false, false) + "Sample Profile loader", false, false) INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker) INITIALIZE_PASS_END(SampleProfileLoaderLegacyPass, "sample-profile", - "Sample Profile loader", false, false) + "Sample Profile loader", false, false) bool SampleProfileLoader::doInitialization(Module &M) { auto &Ctx = M.getContext(); |
