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

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

Thanks for bringing this up. Can you make this optional under a switch? Should be on by default. Currently without a switch we have to go a round trip (ext->txt->ext) to get rid of the symbol list in an extbinary profile.



================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:108
+  }
+  Writer->setProfileSymbolList(&SymbolList);
+
----------------
Move this into the if block above?


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

https://reviews.llvm.org/D110859