[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