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

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

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


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44247