[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