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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits