[PATCH] D35894: [clangd] Code hover for Clangd

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added a comment.

In https://reviews.llvm.org/D35894#829190, @malaperle wrote:

> @Nebiroth I think it's OK to put this on hold until we make the "semantic" hover and figure out how to have both. From our perspective, this is going beyond parity of what we had before but it's seems like the right thing to do.


BTW, just noticed that VS Code shows the range reported by `findDefinitions` when you hover over a symbol and holding Ctrl.
Would calling `findDefinitions` help you to do the same thing? That would allow to get closer to feature parity with CDT without implementing semantic hover in clangd.


https://reviews.llvm.org/D35894