[PATCH] D95056: [CSSPGO] LTO option for pseudo probe

[Wenlei He via Phabricator via llvm-commits]

wenlei accepted this revision.
wenlei added a comment.
This revision is now accepted and ready to land.

lgtm with minor comment.



================
Comment at: lld/ELF/Options.td:553
+def lto_pseudo_probe_for_profiling: F<"lto-pseudo-probe-for-profiling">,
+  HelpText<"Encode pseudo probes for sample profiling.">;
 def disable_verify: F<"disable-verify">;
----------------
hoy wrote:
> wenlei wrote:
> > nit: Encode -> Emit? the clang driver help messages says "Emit pseudo probes for sample profiler", would be good change to "profiling" there too.
> Good catch, changed Encode and profiling.
I actually meant to suggest both lld and clang driver use "Emit pseudo probes for sample profiling", the switch name uses profiling. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95056