[PATCH] D54061: [clang-tidy] run() doesn't update the SourceManager.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 6 01:32:30 PST 2018

This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE346219: [clang-tidy] run() doesn't update the SourceManager. (authored by sammccall, committed by ).

Changed prior to commit:

  rCTE Clang Tools Extra



Index: clang-tidy/ClangTidy.cpp
--- clang-tidy/ClangTidy.cpp
+++ clang-tidy/ClangTidy.cpp
@@ -441,7 +441,9 @@
 void ClangTidyCheck::run(const ast_matchers::MatchFinder::MatchResult &Result) {
-  Context->setSourceManager(Result.SourceManager);
+  // For historical reasons, checks don't implement the MatchFinder run()
+  // callback directly. We keep the run()/check() distinction to avoid interface
+  // churn, and to allow us to add cross-cutting logic in the future.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54061.172719.patch
Type: text/x-patch
Size: 568 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181106/90363384/attachment.bin>

More information about the cfe-commits mailing list