[PATCH] D131046: [clang-tools-extra]Replace find/insert with try_emplace

[Adrian Vogelsgesang via Phabricator via cfe-commits]

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