[PATCH] D45619: [Time-report] (1) Use special new Clang flag 'FrontendTimesIsEnabled' instead of 'llvm::TimePassesIsEnabled' inside -ftime-report feature

Andrew V. Tischenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 13 05:56:09 PDT 2018


avt77 created this revision.
avt77 added reviewers: mgorny, russell.gallop, efriedma, rsmith, thakis, davezarzycki, RKSimon, simon.f.whittaker.

To simplify the review and commit of D43578 <https://reviews.llvm.org/D43578> I decided to spilt it in several small parts.

This patch is the first part of the new series of patches. This patch introduces a dedicated boolean to deal with -ftime-report Clang switch (instead of 'llvm::TimePassesIsEnabled' which was LLVM dependent and potentionally could increase Clang compilation time).

Next patch will show usage of llvm::NamedRegionTimer instead of llvm::TimeRegion - it will really improve the generating output of the given feature.

Then I'll show the new class to deal with reqursive time counters (instead of approach shown in CodeGenAction.cpp).

And then we'll try new time counters (Preprocessor, Include Files, Parsing, Sema, CodeGen, etc.) The final list of counters will grow in dpendence of possible future requirements.

Finally, I'm going to introduce the feature which is similar to this one <https://aras-p.info/blog/2017/10/23/Best-unknown-MSVC-flag-d2cgsummary/>.

Hope, all these efforts could be interesting for many of us.


https://reviews.llvm.org/D45619

Files:
  include/clang/Frontend/Utils.h
  lib/Basic/CMakeLists.txt
  lib/Basic/FrontendTiming.cpp
  lib/CodeGen/BackendUtil.cpp
  lib/CodeGen/CodeGenAction.cpp
  test/Frontend/ftime-report-template-decl.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45619.142383.patch
Type: text/x-patch
Size: 8971 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180413/5214be10/attachment.bin>


More information about the cfe-commits mailing list