[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