[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:26:53 PDT 2019


anton-afanasyev marked an inline comment as done.
anton-afanasyev added a subscriber: aras-p.
anton-afanasyev added inline comments.


================
Comment at: clang/docs/ClangCommandLineReference.rst:1943-1947
 .. option:: -ftime-report
 
+.. option:: -ftime-trace
+
+Turn on time profiler
----------------
lebedev.ri wrote:
> lebedev.ri wrote:
> > While there add a description to `-ftime-report` and document how they are different?
> I mean, state that one is chrome json trace and another is console table output
The difference is actually deeper at the moment. `-ftime-report` output is buggy, short and uninformative (as noted by @aras-p in his blog post http://aras-p.info/blog/2019/01/12/Investigating-compile-times-and-Clang-ftime-report/). I would like just to remove `-ftime-report` option as obsolete and unsupported.


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