[PATCH] D62839: [clangd] Index API and implementations for relations

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 12 20:16:56 PDT 2019


nridge marked an inline comment as done.
nridge added inline comments.


================
Comment at: clang-tools-extra/clangd/index/Index.h:77
+struct RelationsRequest {
+  SymbolID Subject;
+  index::SymbolRole Predicate;
----------------
kadircet wrote:
> sorry for missing it in previous iteration. but this should also enable batch queries. let's make this one a `llvm::DenseMap<SymbolID>`.
> And in the callback we should output both the `SymbolID` of the `Subject` and `Object`.
> 
> We have a network based index(internally) and it uses this batch query optimization to get rid of network latency.
What is an example use case for a batch query? In a typical LSP editor, the user can only request a type hierarchy for one class at a time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62839





More information about the cfe-commits mailing list