[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