[clang] [clang-tools-extra] [clang-tidy] [Modules] Skip checking decls in clang-tidy (PR #145630)

[Balazs Benics via cfe-commits]

================
@@ -1469,6 +1470,12 @@ bool MatchASTVisitor::TraverseDecl(Decl *DeclNode) {
     return true;
   }
 
+  if (Options.SkipDeclsInModules && DeclNode->isFromASTFile()) {
+    auto *M = DeclNode->getOwningModule();
+    if (M && (M->isInterfaceOrPartition() || M->isGlobalModule()))
+      return true;
+  }
+
----------------
steakhal wrote:

I'm not familiar with modules at all, I just want to make sure we check this the right way as there are about 8 member functions matching the `is*Module()` pattern.

https://github.com/llvm/llvm-project/pull/145630