[PATCH] D43578: -ftime-report switch support in Clang

[Andrew V. Tischenko via Phabricator via llvm-commits]

avt77 added a comment.

In https://reviews.llvm.org/D43578#1016751, @MatzeB wrote:

> - Just a warning: I'm a bit skeptical timers will work reliable for things that happen in well under a millisecond (did you do some sanity checking? i.e. do all the timer roughly add up to the time spent in the frontend?)
> - In the same vain I wonder if you add overhead by adding timers to functions that are likely called hundreds of times for a typical source file. Have you measured a release build with and without your patch.




1. You can see in the report very small numbers - it seems eveything works properly. But maybe you have some concrect examples to check?
2. I'm going to build bootsraps with and w/o timers - I'll be back with results when it's done.

But the main idea of this version of the patch was to get understanding about interesting timers: what exactly do we want to see in report? Parser only? Sema? Some special parts of Clang Code Gen?...


https://reviews.llvm.org/D43578