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

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:35
 
+static cl::opt<bool> PopulateProfileSymbolList(
+    "populate-profile-symbol-list", cl::init(true), cl::Hidden,
----------------
On the second thought, do we want the default value of `PopulateProfileSymbolList` to be true or false? We probably do not want the list for either CS or non-CS in practice for now.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:101
+    for (const auto &Item : ProfileMap) {
+      const FunctionSamples Profile = Item.second;
+      SymbolList.add(Profile.getName(), true);
----------------
Use const FunctionSamples& ?


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

https://reviews.llvm.org/D110859