[PATCH] D50896: [clangd] Add xrefs LSP boilerplate implementation.

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added inline comments.


================
Comment at: clangd/XRefs.cpp:665
+std::vector<Location> references(ParsedAST &AST, Position Pos,
+                                 bool IncludeDeclaration,
+                                 const SymbolIndex *Index) {
----------------
Are we going to support the `IncludeDeclaration` option?
What should its semantics be?

I'm trying to figure out a better name for it, can't get what should it do by looking at the code at the moment :-)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D50896