[PATCH] D36847: [Support] Add reentrant start/stop Timer methods

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 09:57:51 PDT 2017


MatzeB added a comment.

In https://reviews.llvm.org/D36847#850238, @modocache wrote:

> Replace `Running` member with `StartCount`.
>
> > Looking at https://reviews.llvm.org/D36946 I'm not convinced yet that timer is a good idea there at all.
>
> Oh! Could you explain why you feel this way?


Sorry, I meant https://reviews.llvm.org/D36492 which I commented on.


https://reviews.llvm.org/D36847





More information about the llvm-commits mailing list