[PATCH] D41461: Add hasProfileData() to check if a function has profile data. NFC.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 17:16:33 PST 2017


eraman added a comment.

In https://reviews.llvm.org/D41461#962311, @davidxl wrote:

> LGTM
>
> Perhaps change the new interface documentation slightly to reflect that it is for real profile data.


Expanded the comments to indicate that this returns true if entry counts from a profile run are present. Once I add synthetic counts, I can make it clear  that we are looking for real counts.

Will submit with this comment change.


https://reviews.llvm.org/D41461





More information about the llvm-commits mailing list