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

[Marc-Andre Laperle via Phabricator via cfe-commits]

malaperle added a comment.

In https://reviews.llvm.org/D35894#830178, @ilya-biryukov wrote:

> 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.


That looks very interesting, I had no idea. We'll have to try that. Thanks!


https://reviews.llvm.org/D35894