[PATCH] D85840: [OpenMPOpt][SplitMemTransfer] Is it possible/worth it to split the runtime call?

[Stefan Stipanovic via Phabricator via llvm-commits]

sstefan1 added a comment.

Can you add parent revisions for this and maybe other related patches? It will be easier to know the order.



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:948
 
-    if (auto *I = canBeMovedUpwards(MT)) {
-      // TODO: Split call and move "issue" below I.
+    if (canBeMovedUpwards(MT) || canBeMovedDownwards(MT)) {
+      // TODO: Split runtime call.
----------------
If I'm not wrong `canBeMovedDownwards()` never returns `nullptr`?


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:1005
+  CallInst *RC = MT.RuntimeCall;
+  auto *I = RC->getNextNode();
+  while (I) {
----------------
I'd put `Instruction *` here.
Also, Maybe put it in the while condition?


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

https://reviews.llvm.org/D85840