sammccall added a comment. In https://reviews.llvm.org/D44247#1031345, @malaperle wrote:
> I was going to change the symbol index to do the opposite :) The range of > definitions including the bodies of functions, etc is used in a "peek > definition" feature by several LSP clients. So for example in VSCode, you can > hold Ctrl and hover on a function call and see its definition in a popup. > There was some discussion about this in https://reviews.llvm.org/D35894 Interesting - that seemed like a more natural interpretation of LSP to me. Others talked me down, motivated (I think) by nicer behavior of jump-to-definition... will bring it up again :) Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D44247 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
