[PATCH] D58675: [clang] Adds `-ftime-trace` option to clang that produces Chrome `chrome://tracing` compatible JSON profiling output dumps
Anton Afanasyev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 29 12:57:56 PDT 2019
anton-afanasyev added a comment.
In D58675#1448112 <https://reviews.llvm.org/D58675#1448112>, @rnk wrote:
> lgtm
>
> I think this is ready. We can adjust the overloads after the fact. I'd like to get the feature in so we can make improvements independently.
Ok, so I'm to commit this and fix three points in the next commit(s):
1. Improve hashing (it's easy).
2. Use json utility.
3. Improve function_ref constructor and delete StringRef("...") explicit casting.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58675/new/
https://reviews.llvm.org/D58675
More information about the cfe-commits
mailing list