[PATCH] D78310: [ProfileSummary] Add partial profile annotation on IR.

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 20 10:48:31 PDT 2020


yamauchi accepted this revision.
yamauchi added a comment.
This revision is now accepted and ready to land.

Nice script.

Would it be also an option to make the IsPartialProfile MD node optional (by making getFromMD a bit more flexible and treating an absence as "0")? There may be a need to add more fields in the future.

Not a blocker, though.

LGTM.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D78310





More information about the llvm-commits mailing list