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

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 17 02:32:25 PST 2019


kadircet marked 2 inline comments as done.
kadircet 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;
----------------
ilya-biryukov wrote:
> 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.
sure filed https://github.com/clangd/clangd/issues/229


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