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
