[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