[PATCH] D66850: Do not attempt visit child nodes of NULL type

[Bert Belder via Phabricator via cfe-commits]

piscisaureus created this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D66850

Files:
  clang/include/clang/AST/ASTNodeTraverser.h


Index: clang/include/clang/AST/ASTNodeTraverser.h
===================================================================
--- clang/include/clang/AST/ASTNodeTraverser.h
+++ clang/include/clang/AST/ASTNodeTraverser.h
@@ -130,9 +130,9 @@
 
   void Visit(const Type *T) {
     getNodeDelegate().AddChild([=] {
-      getNodeDelegate().Visit(T);
       if (!T)
         return;
+      getNodeDelegate().Visit(T);
       TypeVisitor<Derived>::Visit(T);
 
       QualType SingleStepDesugar =



Index: clang/include/clang/AST/ASTNodeTraverser.h
===================================================================
--- clang/include/clang/AST/ASTNodeTraverser.h
+++ clang/include/clang/AST/ASTNodeTraverser.h
@@ -130,9 +130,9 @@
 
   void Visit(const Type *T) {
     getNodeDelegate().AddChild([=] {
-      getNodeDelegate().Visit(T);
       if (!T)
         return;
+      getNodeDelegate().Visit(T);
       TypeVisitor<Derived>::Visit(T);
 
       QualType SingleStepDesugar =

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits