[PATCH] D58341: [clangd] Index UsingDecls
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 18 07:42:46 PST 2019
ilya-biryukov added a comment.
For context: @hokein mentioned problems in the clangd's code completion if we would index these symbols.
This patch in itself does not hurt much, users of the indexing API can decide how to deal with `UsingDecl` on their own, clangd is just one of the clients.
> I wonder how does merge work with Sema results? See the case below, IIUC our indexer has one symbol for this using decl, but the code completion result from Sema contains two symbols. The symbol ids of these 3 symbols are different, so we will end up with 3 completion results.
That's true, but we're not sure how much this would hurt in practice. Currently we don't show any results from dynamic index for `std::strcmp`, which is arguably worse than showing an extra completion item for the using.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58341/new/
https://reviews.llvm.org/D58341
More information about the cfe-commits
mailing list