[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