[PATCH] D47196: [Time-report ](2): Recursive timers in Clang

Andrew V. Tischenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 30 02:08:03 PDT 2018


avt77 added a comment.

Hi All,
What should I do to simplify the review?
I could remove all LLVM_DEBUG related stuff; I could remove all addtional counters and leave only necessary one or two of them. As result the patch will become shorter.

Should I do all these truncations to simplify the review?
Or maybe I should add comments and/or docs in Utils.h describing the new infrastructure?
Maybe you need examples of the produced output?


https://reviews.llvm.org/D47196





More information about the cfe-commits mailing list