[PATCH] D45717: [clangd] Using index for GoToDefinition.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 23 02:16:34 PDT 2018


hokein added a comment.

I have updated the patch according to offline discussion -- for each symbol, we will return both decl and def locations (if available, def first) as they seems to be most sensible to users. We always prefer location from AST over Index when conflicts.



================
Comment at: clangd/XRefs.cpp:261
+
+  for (auto It : ResultCandidates)
+    Result.push_back(It.second);
----------------
sammccall wrote:
> Here we're returning exactly one candidate per symbol we identified as a candidate. There are other choices - why this one?
Yeah, this was one **definition** location per  symbol, made a change according to our discussion.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D45717





More information about the cfe-commits mailing list