kadircet added inline comments.

================
Comment at: clang-tools-extra/clangd/FindTarget.cpp:702
           continue;
+        if (!D.getField())
+          continue;
----------------
thanks for fixing this !

But I suppose we should rather return a result with empty `TargetDecl` as this 
is what API promises:

`  /// For dependent and unresolved references, Targets can also be empty.`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78181



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

Reply via email to