HomePhabricator

[ProfileSummary] Count callsite samples when computing total samples.

Description

[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

Details

Committed
twohApr 3 2019, 12:54 PM
Differential Revision
D59835: [ProfileSummary] Count callsite samples when computing total samples.
Parents
rL357626: Fix and simplify PrepareCommandsForSourcing
Branches
Unknown
Tags
Unknown