[PATCH] D43578: -ftime-report switch support in Clang
Andrew V. Tischenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 16 01:51:45 PDT 2018
avt77 added a comment.
In https://reviews.llvm.org/D43578#1068417, @rsmith wrote:
> I think we need to fix the overlap issue as a prerequisite to adding timers with large amounts of overlap, especially self-overlap. Otherwise the numbers will likely do more harm than good, as they will significantly misattribute runtime. Fortunately, I think that should only require some relatively small changes to the LLVM timer infrastructure.
JFYI: the given patch introduces non-recursive (self-overlaps) timers only. I'm going to introduce self-overlaps timers when https://reviews.llvm.org/D45619 is committed.
https://reviews.llvm.org/D43578
More information about the cfe-commits
mailing list