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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
