[PATCH] D71545: [clangd] Improve hover for auto on template instantiations

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 17 02:13:36 PST 2019


ilya-biryukov added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/HoverTests.cpp:556
+            // FIXME: Drop default arguments.
+            HI.Name = "Foo<X, int>";
+            HI.Kind = index::SymbolKind::Class;
----------------
NIT: Maybe remove default argument from this test for now and file a bug for fixing this instead?
Tracking this in the issue tracker seems to be better than in code.


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