[PATCH] D36847: [Support] Add "reference-counted" Timer

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 14:13:05 PDT 2017


vsk added subscribers: davide, rafael.
vsk added a comment.

I tried this a while back: https://reviews.llvm.org/D20735

I ended up abandoning the patch because some some folks weren't totally comfortable with a timer that can be started twice. I think we're seeing that this is a use case that's becoming more important. Actually, @rafael mentioned that he's OK with ref-counted timers "if it is cumbersome to use a non ref counted timer in the current users", and it looks like that's the case. @davide, wdyt of revisiting the ref-counted timer decision?


https://reviews.llvm.org/D36847





More information about the llvm-commits mailing list