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

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 7 19:53:29 PDT 2026


================
@@ -732,7 +732,8 @@ static bool findArgParts(Argument *Arg, const DataLayout &DL, AAResults &AAR,
     return true; // No users, this is a dead argument.
 
   // Sort parts by offset.
-  append_range(ArgPartsVec, ArgParts);
+  for (const auto &Part : ArgParts)
+    ArgPartsVec.emplace_back(Part.first, Part.second);
----------------
kazutakahirata wrote:

Could we use structured bindings here?

```cpp
for (const auto &[Offset, Part] : ArgParts)
  ArgPartsVec.emplace_back(Offset, Part);
```


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


More information about the llvm-commits mailing list