[clang] [alpha.webkit.UncountedLocalVarsChecker] Handle null Type in CanTriviallyDestruct (PR #182129)

[Ryosuke Niwa via cfe-commits]

================
@@ -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