[PATCH] D86474: [OpenMPOpt][SplitMemTransfer] Grouping the setup instructions for the runtime call.

[Stefan Stipanovic via Phabricator via llvm-commits]

sstefan1 added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:511
+    // Get setup instruction for %offload_baseptrs.
+    if (!getSetupInstructions(OffloadArrays[0]))
+      return false;
----------------
I think it would be useful to make these constants with proper names, eg. `constexpr int OffloadPtrsIdx = 0;` or something similar. WDYT?


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

https://reviews.llvm.org/D86474