[PATCH] D59641: [clangd] Show template argument list in workspacesymbols and documentsymbols responses

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 11 02:00:03 PDT 2019


ilya-biryukov added a comment.

LG, just have a simple clarifying question.



================
Comment at: clang-tools-extra/clangd/AST.cpp:112
   if (!Out.str().empty()) {
-    // FIXME(ibiryukov): do not show args not explicitly written by the user.
-    if (auto *ArgList = getTemplateSpecializationArgs(ND))
----------------
Where did this FIXME go? Is this now fixed?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59641/new/

https://reviews.llvm.org/D59641





More information about the cfe-commits mailing list