[PATCH] D68601: [SampleFDO] Add indexing for function profiles so they can be loaded on demand in ExtBinary format

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 8 15:45:26 PDT 2019


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

LGTM. Thanks!

> Symbol list can be provided to llvm-profdata in a plain text file.

I thought it's more convenient to have PSL auto-populated by the tool that generates AutoFDO profile, or is there any reason for not using auto-generated PSL, and instead providing a plain text file as side input?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68601





More information about the llvm-commits mailing list