avogelsgesang added inline comments.
================ Comment at: clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp:140 + // This replacement is a duplicate of one suggested by another TU. + if (!InsertPos.second) { return; ---------------- Previously, we had an additional check `It->second != SourceTU` here. I don't understand what this check was doing, but this does not seem to be a pure refactoring. If this is still an NFC change, please update the commit message to explain why this condition was unneeded Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131046/new/ https://reviews.llvm.org/D131046 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits