[PATCH] D65202: [Support] Fix `-ftime-trace-granularity` option

Anton Afanasyev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 24 06:38:35 PDT 2019


anton-afanasyev marked 2 inline comments as done.
anton-afanasyev added inline comments.


================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:1772
   Opts.TimeTrace = Args.hasArg(OPT_ftime_trace);
+  Opts.TimeTraceGranularity =
+      getLastArgIntValue(Args, OPT_ftime_trace_granularity_EQ, 500, Diags);
----------------
sammccall wrote:
> You've got the default repeated here. Is it possible to set this conditionally here, or use the existing value as default like
> 
> `Opts.TTG = getLastArgIntValue(Args, OPT_fttg_EQ, Opts.TTG, Diags)`
Ok, done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65202





More information about the cfe-commits mailing list