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

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added a comment.

- 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.


https://reviews.llvm.org/D43578