[PATCH] D44247: [clangd] Use identifier range as the definition range.

Marc-Andre Laperle via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 8 07:10:06 PST 2018

malaperle added a comment.

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

  rCTE Clang Tools Extra


More information about the cfe-commits mailing list