aeubanks added inline comments.

================
Comment at: llvm/tools/opt/NewPMDriver.cpp:136-138
+static cl::opt<bool> PseudoProbeForProfiling(
+    "new-pm-pseudo-probe-for-profiling", cl::init(false), cl::Hidden,
+    cl::desc("Emit pseudo probes to enable PGO profile generation."));
----------------
hoy wrote:
> dblaikie wrote:
> > I guess this should probably have some separate testing, if it's a separate 
> > flag/feature? (& flag+tests could be in a separate commit)
> I'm not sure there's a separate need for this switch except for being tested 
> in `unique-internal-linkage-names.ll`. The point of this whole patch is to 
> place the unique name pass before the pseudo probe pass and test it works. 
> Hence it sounds appropriate to me to include all changes in one patch. What 
> do you think?
+1 to hoy's comment. I don't think there's a need to make patches strictly as 
incremental as possible if they're already small. (I would have been okay with 
keeping the Clang change here FWIW)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93656

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to