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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to