[PATCH] D91721: [clangd] textDocument/implementation (LSP layer)

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet added inline comments.
Herald added projects: clang-tools-extra, All.


================
Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:1306
+        for (const LocatedSymbol &Sym : *Overrides)
+          Impls.push_back(Sym.PreferredDeclaration);
+        return Reply(std::move(Impls));
----------------
any particular reason for preferring declarations here that I am missing (in presence of a definition)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91721