[PATCH] D71652: [clangd] Replace shortenNamespace with getQualification
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 2 06:14:28 PST 2020
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: clang-tools-extra/clangd/AST.cpp:307
+ llvm::raw_string_ostream OS(Result);
+ auto Decls = explicitReferenceTargets(
+ ast_type_traits::DynTypedNode::create(QT), DeclRelation::Alias);
----------------
This is such a terrible hack... Could you put a FIXME mentioning this should be done when printing names of the underlying decls instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71652/new/
https://reviews.llvm.org/D71652
More information about the cfe-commits
mailing list