[PATCH] D108556: [clangd] Don't highlight ObjC `id` and `instancetype`

[David Goldman via Phabricator via cfe-commits]

dgoldman marked 2 inline comments as done.
dgoldman added inline comments.


================
Comment at: clang-tools-extra/clangd/FindTarget.cpp:188
     if (const TypedefNameDecl *TND = dyn_cast<TypedefNameDecl>(D)) {
+      if (shouldSkipTypedef(TND))
+        return;
----------------
kadircet wrote:
> why do we need this despite bailing out in the visitor?
Removed, not necessary


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108556

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