[PATCH] Add flag --enable-check-profile to clang-tidy.

Samuel Benzaquen sbenza at google.com
Thu Oct 23 09:06:21 PDT 2014


================
Comment at: clang-tidy/ClangTidy.cpp:220
@@ +219,3 @@
+  ast_matchers::MatchFinder::MatchFinderOptions FinderOptions;
+  if (auto *P = Context.getCheckProfileData()) {
+    FinderOptions.CheckProfiling.emplace(P->Records);
----------------
alexfh wrote:
> nit: No braces around single-statement if bodies in LLVM style.
done

================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.h:198
@@ -183,1 +197,3 @@
 
+  llvm::StringMap<llvm::TimeRecord> CheckProfileData;
+
----------------
alexfh wrote:
> Unused?
Oops. Remnants of a previous implementation.

================
Comment at: clang-tidy/tool/ClangTidyMain.cpp:287
@@ -239,1 +286,3 @@
   printStats(Stats);
+  if (EnableCheckProfile) printProfileData(Profile, llvm::errs());
+
----------------
alexfh wrote:
> nit: Move the if body to the next line.
Fixed.

http://reviews.llvm.org/D5937






More information about the cfe-commits mailing list