hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land.
Thanks, the title is a bit confusing, the modified code is about the index, not XRefs. ================ Comment at: clang-tools-extra/clangd/index/Merge.cpp:116 // files. - More |= Static->refs(Req, [&](const Ref &O) { if (DynamicIndexFileURIs.count(O.Location.FileURI)) ---------------- I assume the potential UB is this code? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70225/new/ https://reviews.llvm.org/D70225 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits