nridge added a comment.

In D71345#1783587 <https://reviews.llvm.org/D71345#1783587>, @sammccall wrote:

> Indeed, sorry - I meant that if we incorporated it into an LSP server, 
> there'd be no way to target it in methods that take a position rather than a 
> selection (go to defn, hover etc).


Ah, right, I forgot that LSP tends to send positions rather than ranges as 
inputs. (I hope that changes in the future, e.g. an issue 
<https://github.com/microsoft/language-server-protocol/issues/377> is on file 
for hover.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71345/new/

https://reviews.llvm.org/D71345



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to