ilya-biryukov added inline comments.
================ Comment at: clangd/XRefs.cpp:201 std::vector<MacroDecl> MacroInfos = DeclMacrosFinder->takeMacroInfos(); + if (!MacroInfos.empty()) { + for (auto Item : MacroInfos) { ---------------- I wonder whether we should fix the `DeclrationAndMacrosFinder` to not return declarations coming from macro instantiations instead. There are other clients (e.g. document highlights) that will probably break in the same way. Do you think it would be possible and it would make sense for `DeclrationAndMacrosFinder` to only return a macro in this case and not return Decls coming from macro expansions? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D44293 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits