[PATCH] D86300: [OpenMPOpt][SplitMemTransfer] Getting values stored in offload arrays

[Hamilton Tobon-Mosquera via Phabricator via llvm-commits]

hamax97 added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:787
+  /// \p RuntimeCall into the offload arrays in \p OAs.
+  bool getValuesInOffloadArrays(CallInst &RuntimeCall, OffloadArrayPtr OAs[3]) {
+    // A runtime call that involves memory offloading looks something like:
----------------
jdoerfert wrote:
> Don't you need a reference here? We also need a test, maybe just one that prints the result?
If I pass an array it's passed as a pointer, isn't it?.

A unittest?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86300