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

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 3 02:14:12 PST 2018


hokein added a comment.

With more test, it turns out that `ASTNode.OrigD` is not always pointed to the definition, so we can't rely on `D` or `ASTNode.OrigD` :(.

I revised the way of checking definition, it should works for major cases. Please review it again.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41661





More information about the cfe-commits mailing list