[PATCH] D58675: [clang] Adds `-ftime-trace` option to clang that produces Chrome `chrome://tracing` compatible JSON profiling output dumps
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Mar 30 22:40:20 PDT 2019
anton-afanasyev added a comment.
In D58675#1449051 <https://reviews.llvm.org/D58675#1449051>, @thakis wrote:
> Looks like this landed without tests. Please add tests in a follow-up.
>
> Also, it looks like the flag is currently a cc1 flag. This should probably be a CoreOption instead so that both the gcc-style and the cl-style drivers expose it. Right now, users can't really use this if I read the patch right (without using -Xclang, which is discouraged.)
Ok, thanks, I'm to add tests in a follow-up.
It works like `clang -ftime-trace main.cpp` now. Not sure I know where CoreOption is, but I'm to figure out it, thanks.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58675/new/
https://reviews.llvm.org/D58675
More information about the llvm-commits
mailing list