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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to