================
@@ -1507,11 +1544,17 @@ bool MatchASTVisitor::TraverseStmt(Stmt *StmtNode,
DataRecursionQueue *Queue) {
}
bool MatchASTVisitor::TraverseType(QualType TypeNode) {
+ if (shouldSkipNode(TypeNode))
----------------
carlosgalvezp wrote:
My goal was to establish a somewhat consistent check for all the `Traverse*`
functions that could be extended in the future, like I said there's a PR about
modules where this could fit well.
But if it's deemed premature generalization/overkill I can a) remove it
altogether or b) overload the function for `QualType` and directly return
`true`, skipping the logic that ends up checking the default-constructed
`SourceLocation` for being in a system header. Let me know! :)
https://github.com/llvm/llvm-project/pull/151035
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
