[PATCH] D115950: [llvm-profgen] Use shared string table for function name to save memory usage

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

Thanks for the change. How much space do you see this helps save?



================
Comment at: llvm/lib/MC/MCPseudoProbe.cpp:354
 
+    auto It = FuncNameStrings.insert(Name.str());
     // Initialize PseudoProbeFuncDesc and populate it into GUID2FuncDescMap
----------------
wenlei wrote:
> Just double checking, so the underlying data for probe section is freed after decoding? Where does that happen? 
The real name is on the disk. When read, a temp object `ErrorOrName` is created which is then freed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115950