[PATCH] D61025: Add an optional arg to include/exclude synthetic profile when query hasProfileData

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 23 16:08:50 PDT 2019


eraman added a comment.

Instead of passing a boolean, would it be better to pass ProfileCountType enum? Right now it doesn't make a difference as there are only two valid profile count types, but if that set expands, we could make them like bitmasks and then get a subset match.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61025/new/

https://reviews.llvm.org/D61025





More information about the llvm-commits mailing list