This is an archive of the discontinued LLVM Phabricator instance.

[ProfileSummary] Make getProfileCount a non-static member function.
ClosedPublic

Authored by eraman on May 9 2017, 11:36 AM.

Details

Summary

This change is required because the notion of count is different for sample profiling and getProfileCount will need to determine the underlying profile type.

Diff Detail

Repository
rL LLVM

Event Timeline

eraman created this revision.May 9 2017, 11:36 AM
This revision is now accepted and ready to land.May 9 2017, 1:50 PM
danielcdh edited edge metadata.May 9 2017, 2:38 PM

Thanks for the review.

danielcdh accepted this revision.May 9 2017, 2:39 PM

oops, sent to the wrong patch review...

This patch LGTM.

eraman added a comment.May 9 2017, 4:21 PM

In offline discussion, Dehao asked me to pass PSI when constructing InlineFunctionInfo from PartialInliner. Will submit this patch with that change.

This revision was automatically updated to reflect the committed changes.