]> granicus.if.org Git - llvm/commit
[ProfileSummary] Count callsite samples when computing total samples.
authorTaewook Oh <twoh@fb.com>
Wed, 3 Apr 2019 19:54:43 +0000 (19:54 +0000)
committerTaewook Oh <twoh@fb.com>
Wed, 3 Apr 2019 19:54:43 +0000 (19:54 +0000)
commit172ff13310aecd0e194eabcbc4a910888682815b
tree435430e9646b7314d5b450331a82c74e9a2a37c1
parent5c7b7d7261b1b9f0e5d7b3a8bf0ed822b1ee3c04
[ProfileSummary] Count callsite samples when computing total samples.

Summary: Currently ProfileSummaryBuilder doesn't count into callsite samples when computing total samples. Considering that ProfileSummaryInfo is used to checked the hotness of not only body samples but also callsite samples (from SampleProfileLoader), I think the callsite sample counts should be considered when computing total samples.

Reviewers: eraman, danielcdh, wmi

Subscribers: hiraditya, jdoerfert, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D59835

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@357627 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/ProfileData/ProfileCommon.h
lib/ProfileData/ProfileSummaryBuilder.cpp
test/Transforms/SampleProfile/Inputs/inline.compactbinary.afdo
test/Transforms/SampleProfile/Inputs/summary.prof
test/Transforms/SampleProfile/compact-binary-profile.ll
test/Transforms/SampleProfile/summary.ll