[PATCH] D83177: [llvm-reduce] Reducing call operand bundles

Tyker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 13:01:47 PDT 2020


Tyker added a comment.

Thank you for reducing rG7ea46aee3670981827c04df89b2c3a1cbdc7561b <https://reviews.llvm.org/rG7ea46aee3670981827c04df89b2c3a1cbdc7561b>
this seems quite useful.



================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceOperandBundles.cpp:138
+
+  for_each(R.CallsToRefine, [](const auto &P) {
+    return maybeRewriteCallWithDifferentBundles(P.first, P.second);
----------------
Maybe we should make CallsToRefine a MapVector since the association from a index to a Bundle depends on its order in the map.
and the key depends on a pointer value that will change when the Module gets cloned.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83177/new/

https://reviews.llvm.org/D83177





More information about the llvm-commits mailing list