[PATCH] D110859: [llvm-profgen] Support symbol list for accurate profile

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 1 17:25:34 PDT 2021


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

LGTM. We probably need this for CS profile too.



================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:105
+  // only support for non-CS profile.
+  ProfileSymbolList SymbolList;
+  if (PopulateProfileSymbolList && OutputFormat == SPF_Ext_Binary) {
----------------
This can be moved into the if block too.


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

https://reviews.llvm.org/D110859



More information about the llvm-commits mailing list