[clang-tools-extra] [clangd] Guard against null TemplateName in DumpVisitor (PR #181554)

[via cfe-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clangd

Author: Emily Dror (edror12)

<details>
<summary>Changes</summary>

Add a guard against null values for TemplateName in DumpVisitor::TraverseTemplateName.

Fixes: #<!-- -->180902

---
Full diff: https://github.com/llvm/llvm-project/pull/181554.diff


1 Files Affected:

- (modified) clang-tools-extra/clangd/DumpAST.cpp (+2-2) 


``````````diff
diff --git a/clang-tools-extra/clangd/DumpAST.cpp b/clang-tools-extra/clangd/DumpAST.cpp
index cd409a2b930ef..2b673128493b3 100644
--- a/clang-tools-extra/clangd/DumpAST.cpp
+++ b/clang-tools-extra/clangd/DumpAST.cpp
@@ -347,8 +347,8 @@ class DumpVisitor : public RecursiveASTVisitor<DumpVisitor> {
     });
   }
   bool TraverseTemplateName(const TemplateName &TN) {
-    return traverseNode("template name", TN,
-                        [&] { Base::TraverseTemplateName(TN); });
+    return TN.isNull() || traverseNode("template name", TN,
+                                       [&] { Base::TraverseTemplateName(TN); });
   }
   bool TraverseTemplateArgumentLoc(const TemplateArgumentLoc &TAL) {
     return traverseNode("template argument", TAL,

``````````

</details>


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