[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