Whitney added inline comments.
================ Comment at: clang/include/clang/Driver/Options.td:2832 +def ftime_trace_path : Joined<["-"], "ftime-trace-path=">, Group<f_Group>, + HelpText<"Path which stores the output files of time profiler">, + Flags<[CC1Option, CoreOption]>, ---------------- jamieschmeiser wrote: > which specifies the output files for -ftime-trace What do you think of "which specifies the output directory for -ftime-trace"? As from your example, this option doesn't specify a file, instead it specifies a directory. ================ Comment at: clang/include/clang/Frontend/FrontendOptions.h:502 + /// Path which stores the output files of time profiler. + std::string TimeTracePath; ---------------- Please update this comment according too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128048/new/ https://reviews.llvm.org/D128048 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
