[PATCH] D20735: [Support] Allow nesting paired calls to {start, stop}Timer

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 10:05:41 PDT 2016


davide added a comment.

In http://reviews.llvm.org/D20735#442381, @rafael wrote:

> How hard would it be to change the callers that currently depend on this?


I have a patch that changes the clang timer where I hit this issue. I'm going to put it up for review.
The real worry on my side is that I'm not aware of all the possible cases where we rely on this behavior.
We can just drop Vedant's patch on the floor, and wait until we hit the assertions, and fix the bugs as they show up.
What's your opinion?


http://reviews.llvm.org/D20735





More information about the llvm-commits mailing list