[PATCH] D71545: [clangd] Improve hover for auto on template instantiations
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 18 00:54:48 PST 2019
sammccall added inline comments.
================
Comment at: clang-tools-extra/clangd/unittests/HoverTests.cpp:1209
// FIXME: Print template instantiation parameters.
- HI.Name = "initializer_list";
+ HI.Name = "initializer_list<int>";
HI.Kind = index::SymbolKind::Class;
----------------
Hang on, I think we're going round in circles with this design.
IIRC the idea was that `Name` doesn't include template parameters, signature, etc, so clients control rendering.
Isn't it easy to reconstitute this from the template argument list in the hover info?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71545/new/
https://reviews.llvm.org/D71545
More information about the cfe-commits
mailing list