[PATCH] D72498: [clangd] Print underlying type for decltypes in hover

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 10 06:36:44 PST 2020


sammccall added a comment.

I think i'm also comfortable with marking the linked bug as wontfix.

It's a contrived example that makes clangd look silly (why decltype(a) instead of int?) but also the user look silly (why hover the variable rather than the decltype?).
Real examples are certainly more mixed.

The only thing that gives me pause is that it does seem to be decltype and auto belong in the same bucket (more so than typedef). If we can make those consistent in typeprinter, I'm happy.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72498





More information about the cfe-commits mailing list