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

2018-04-16 Thread Michael Zolotukhin via Phabricator via cfe-commits
mzolotukhin added a comment. > What kinds of would be useful to you? (How do you want the > runtime of Clang broken down?) Are vertical slices (through all Clang's > various layers and potentially parts of LLVM) -- as this patch will produce > -- useful, or would you really want horizontal sli

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

2018-04-14 Thread Michael Zolotukhin via Phabricator via cfe-commits
mzolotukhin added a comment. Hi, I've been monitoring compile-time for quite a while, so let my put my 2 cents here too. > Who is the audience for this information? > What information do they want from a time report? > How do we present that information in a way that's not misleading (given

[PATCH] D36492: [RFC][time-report] Add preprocessor timer

2017-08-10 Thread Michael Zolotukhin via Phabricator via cfe-commits
mzolotukhin added a comment. FWIW, I strongly support the idea of adding more detailed timers into the frontend. Thanks for working on it! I probably won't be very helpful in reviewing this code, but I'd appreciate if you CC me in the future patches. Thanks, Michael https://reviews.llvm.org/D