[PATCH] D85628: [HotColdSplitting] Add command line options for supplying cold function names via user input.

[Ruijie Fang via Phabricator via llvm-commits]

rjf marked an inline comment as done.
rjf added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/HotColdSplitting.cpp:687
+  // Read in user-defined cold function names, if any.
+  if (ColdFunctionsList != "") {
+    std::stringstream CFStream(ColdFunctionsList);
----------------
rjf wrote:
> vsk wrote:
> > Might be better to use SpecialCaseList.h rather than hand-rolling something new.
> It seems like the public API of `SpecialCaseList.h` only supports looking up whether a particular string is contained in the list, rather than retrieving all strings in the list. If we do this then we'll either have to a) traverse the entire module to match which function names in the current module are in the list, and mark them as cold, or b)  
Sorry, the above comment was written (and published) in mistake. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85628