[PATCH] D128451: [lld, ELF and mac] Add --time-trace=<file>, remove --time-trace-file=<file>

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 23 12:30:14 PDT 2022


thakis added inline comments.


================
Comment at: lld/ELF/Options.td:411
 
-def time_trace: FF<"time-trace">, HelpText<"Record time trace">;
-def time_trace_file_eq: JJ<"time-trace-file=">, HelpText<"Specify time trace output file">;
+def time_trace_eq: JJ<"time-trace=">, MetaVarName<"<file>">,
+  HelpText<"Record time trace to <file>">;
----------------
MaskRay wrote:
> Use `def time_trace`
why? it's an _eq option, no?


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

https://reviews.llvm.org/D128451



More information about the llvm-commits mailing list