[PATCH] D138300: [clangd] Support type hints for `decltype(expr)`

[Nathan Ridge via Phabricator via cfe-commits]

nridge added a comment.

Sorry for being a slow reviewer.

I think showing type hints for `decltype(expr)` would be a nice enhancement. @v1nh1shungry, are you interested in working further on this?

One high-level thought I had is: what if we attached the type hint to the closing `)` of the decltype (and had it pertain only to the `decltype(expr)`, not anything surrounding it like `const` or `&`)? It seems to me that this would both simplify the implementation, and allow us to show hints in places where `decltype` is used in a context unrelated to a variable or function declaration (for example, in something like `using Foo = A<B<decltype(expr), C>, D>`, if `decltype(expr)` was `int`, we would show `using Foo = A<B<decltype(expr) : int,  C>, D>`. What do you think about this approach?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138300