[PATCH] D20748: Handle recursion in LLVMIRGeneration Timer

Davide Italiano via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 8 19:38:44 PDT 2016


I agree with Rafael. I ran with -ftime-report for a couple of weeks on
a lot of tests and never hit this problem.
It actually seems weird we have to handle the timer recursively here
as well (I'd like to avoid it at all), but let's see what Richard has
to say when he gets to this review.

On Wed, Jun 8, 2016 at 10:27 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> Since we found only one user, I think my preference is to handle it there.
>
> Cheers,
> Rafael
>
>
> On 8 June 2016 at 13:49, Vedant Kumar <vsk at apple.com> wrote:
>> vsk added a comment.
>>
>> Ping, any updates on this patch?
>>
>>
>> http://reviews.llvm.org/D20748
>>
>>
>>


More information about the cfe-commits mailing list