[PATCH] D92998: [CSSPGO][llvm-profgen] Pseudo probe based CS profile generation

[Lei Wang via Phabricator via llvm-commits]

wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:43
+    // Enable context-sensitive functionalities in SampleProf
+    FunctionSamples::ProfileIsCS = true;
     if (usePseudoProbes(BinarySampleCounters)) {
----------------
hoy wrote:
> I see. Sounds like the initializations should be moved into `ProfileGenerator::generateProfile` or `ProfileGenerator::write`. What do you think? 
Sounds good! So since it's only used for the writer, I will move them to `ProfileGenerator::write`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92998