ilya-biryukov added inline comments.
================
Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:51
}
+ if (auto *UD = dyn_cast<UsingDecl>(D)) {
+ if (UD->shadow_size() == 0)
----------------
Could we reuse `kindForCandidateDecls` instead?
It's best to keep one way to highlight multiple decls.
================
Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:60
+ });
+ return kindForDecl(Selected->getTargetDecl());
+ }
----------------
Using decls never have any references and we only highlight them at their
declarations locations.
Therefore `kindForDecl` seems like the wrong place for it. Could we add a
`VisitUsingDecl` method to the visitor and have this logic there instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69506/new/
https://reviews.llvm.org/D69506
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
