[PATCH] D66647: [clangd] DefineInline action apply logic with fully qualified names
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 27 02:33:53 PDT 2019
kadircet added a comment.
In D66647#1685554 <https://reviews.llvm.org/D66647#1685554>, @ilya-biryukov wrote:
> Are we planning to fix this right away or should we keep this indefinitely?
> If latter, I believe we should think about some heuristics to avoid qualification in the short term instead.
> If we want to do it right away, that's great! (albeit more complicated, I guess)
That's the end goal, but I am currently working on a patch that will at least strip away the common namespaces
of function declaration and symbols used inside.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66647/new/
https://reviews.llvm.org/D66647
More information about the cfe-commits
mailing list