[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
Tue Mar 26 07:04:09 PDT 2019
anton-afanasyev marked 8 inline comments as done.
anton-afanasyev added inline comments.
================
Comment at: llvm/lib/Support/TimeProfiler.cpp:33
+ switch (C) {
+ case '"':
+ case '\\':
----------------
takuto.ikuta wrote:
> Include escape for '/' too.
> https://tools.ietf.org/html/rfc8259#section-7
Yes, thanks!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58675/new/
https://reviews.llvm.org/D58675
More information about the cfe-commits
mailing list