[PATCH] D43578: -ftime-report switch support in Clang

Andrew V. Tischenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 16 01:51:46 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 llvm-commits mailing list