[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


================
@@ -760,9 +760,8 @@ ParallelToGpuLaunchLowering::matchAndRewrite(ParallelOp parallelOp,
 
   // Now that we succeeded creating the launch operation, also update the
   // bounds.
-  for (auto bound : launchBounds)
-    launchOp.setOperand(getLaunchOpArgumentNum(std::get<0>(bound)),
-                        std::get<1>(bound));
+  for (const auto &bound : launchBounds)
+    launchOp.setOperand(getLaunchOpArgumentNum(bound.first), bound.second);
----------------
kazutakahirata wrote:

How about structured bindings here?
```suggestion
for (const auto &[processor, bound] : launchBounds)
  launchOp.setOperand(getLaunchOpArgumentNum(processor), bound);
```

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


More information about the llvm-commits mailing list