[PATCH] D20748: Handle recursion in LLVMIRGeneration Timer

[Vedant Kumar via cfe-commits]

vsk added a comment.

Comments on this patch -- The increment and decrement snippets seem like they could be pulled into helper methods. That should make it easier to update all users of LLVMIRGeneration. I wasn't able to come up with a regression test, but I do think this patch needs one.

Overall comments -- My main concern is that this isn't the only affected user of the Timer class. I have a preference for http://reviews.llvm.org/D20735 because it defines the underlying problem away. I'll defer to someone else on this (maybe @rafael?).


http://reviews.llvm.org/D20748