[PATCH] D114522: [clangd] Add desugared type to hover

liu hui via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 26 01:18:38 PST 2021


lh123 added inline comments.


================
Comment at: clang-tools-extra/clangd/Hover.cpp:166
+    if (ShouldAKA)
+      AKA = Result + DesugaredTy.getAsString(PP);
+  }
----------------
lh123 wrote:
> It seems we lost `namespace qualifiers`.
> It always display `std::string` as `basic_string<char>`
Should we set `FullyQualifiedName` printing policy  for `DesugaredTy`.


Repository:
  rZORG LLVM Github Zorg

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

https://reviews.llvm.org/D114522



More information about the cfe-commits mailing list