ilya-biryukov added inline comments.
================ Comment at: clangd/XRefs.cpp:137 + // + // FIXME: Exclude declarations from macros. + Decls.clear(); ---------------- NIT: the fixme was a bit hard to follow for me. Maybe make it more clear where the problem should be handled, e.g. ```FIXME: we should avoid adding decls from inside macros in handlDeclOccurence. ``` 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