[Mlir-commits] [clang] [llvm] [mlir] [ADT] Give DenseMapPair its own members instead of a std::pair base. NFC (PR #221853)

Kazu Hirata llvmlistbot at llvm.org
Mon Sep 7 19:53:30 PDT 2026


================
@@ -83,7 +83,7 @@ void PDLPatternModule::mergeIn(PDLPatternModule &&other) {
   for (auto &it : other.configs)
     configs.emplace_back(std::move(it));
   for (auto &it : other.configMap)
-    configMap.insert(it);
+    configMap.try_emplace(it.first, it.second);
----------------
kazutakahirata wrote:

You may not have to break up `it` at all:
```
MethodVFTableLocations.insert_range(NewMethodLocations);
```


https://github.com/llvm/llvm-project/pull/221853


More information about the Mlir-commits mailing list