================
@@ -272,9 +272,9 @@ class RawPtrRefLocalVarsChecker
}
bool TraverseClassTemplateDecl(ClassTemplateDecl *Decl) override {
- if (isSmartPtrClass(safeGetName(Decl)))
- return true;
- return DynamicRecursiveASTVisitor::TraverseClassTemplateDecl(Decl);
+ // We don't traverse primary templates because we only care about
+ // specializations (instantiations).
+ return true;
}
----------------
rniwa wrote:
Oh, I think that code was added to specifically return `true` for smart pointer
classes. I guess your suggestion is to skip analyzing these AST subtrees
altogether?
https://github.com/llvm/llvm-project/pull/182129
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
