[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

[Adrian Prantl via Phabricator via cfe-commits]

aprantl added a comment.

This looks like a good idea — people will probably have some options on the names and descriptions of the statistics  :-)



================
Comment at: clang/lib/Frontend/CompilerInstance.cpp:60
+
+ALWAYS_ENABLED_STATISTIC(NumCompileModule, "Number of compiled modules.");
+
----------------
`NumCompiledModules`?


================
Comment at: clang/lib/Serialization/ASTReader.cpp:151
+                                           "and using cached results.");
+ALWAYS_ENABLED_STATISTIC(NumReadASTCore, "Number of times read AST core.");
+
----------------
`Number of times ReadASTCore() is invoked`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86895/new/

https://reviews.llvm.org/D86895