[PATCH] D93227: [clangd] Smarter hover on auto and decltype

Quentin Chateau via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 17 03:49:23 PST 2020


qchateau added a comment.

> We can't easily add new SymbolKinds (it's an enum we don't own, which we should fix, but that's a yak-shave). But `TypeAlias` almost fits these roles for auto.

Yes I originally wanted to have "deduced-type `auto`" but I realized it would not be easy. I did not think of `TypeAlias`, let's use this until we can do better.

> (There are other cases: function parameter types `void foo(auto X)` and NTTP types `template <auto X> class Y` but we can punt on those)

For now I disabled them (instead of displaying nonsense as we did before). Anyway as a user, I have zero useful information to get when hovering on these word, I prefer nothing rather than a useless pop-up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93227



More information about the cfe-commits mailing list