[PATCH] D41661: [clangd] Don't navigate to forward class declaration when go to definition.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 2 04:54:48 PST 2018


sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clangd/XRefs.cpp:54
     if (isSearchedLocation(FID, Offset))
-      Decls.push_back(D);
+      Decls.push_back(ASTNode.OrigD);
     return true;
----------------
Can you add some motivation here? e.g. the forward decl example


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41661





More information about the cfe-commits mailing list