dgoldman updated this revision to Diff 277103.
dgoldman added a comment.

Move up null check


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83501/new/

https://reviews.llvm.org/D83501

Files:
  clang-tools-extra/clangd/FindTarget.cpp


Index: clang-tools-extra/clangd/FindTarget.cpp
===================================================================
--- clang-tools-extra/clangd/FindTarget.cpp
+++ clang-tools-extra/clangd/FindTarget.cpp
@@ -282,6 +282,8 @@
       D = CID->getCategoryDecl();
       Flags |= Rel::Underlying;
     }
+    if (!D)
+      return;
 
     if (const Decl *Pat = getTemplatePattern(D)) {
       assert(Pat != D);
@@ -290,8 +292,6 @@
       Flags |= Rel::TemplateInstantiation;
     }
 
-    if (!D)
-      return;
     report(D, Flags);
   }
 


Index: clang-tools-extra/clangd/FindTarget.cpp
===================================================================
--- clang-tools-extra/clangd/FindTarget.cpp
+++ clang-tools-extra/clangd/FindTarget.cpp
@@ -282,6 +282,8 @@
       D = CID->getCategoryDecl();
       Flags |= Rel::Underlying;
     }
+    if (!D)
+      return;
 
     if (const Decl *Pat = getTemplatePattern(D)) {
       assert(Pat != D);
@@ -290,8 +292,6 @@
       Flags |= Rel::TemplateInstantiation;
     }
 
-    if (!D)
-      return;
     report(D, Flags);
   }
 
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to