[llvm] [Offload] Copy loaded images into managed storage (PR #158748)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 15 18:18:28 PDT 2025


================
@@ -912,12 +898,12 @@ GenericDeviceTy::loadBinary(GenericPluginTy &Plugin,
 
 #ifdef OMPT_SUPPORT
   if (ompt::Initialized) {
-    size_t Bytes =
-        utils::getPtrDiff(InputTgtImage->ImageEnd, InputTgtImage->ImageStart);
+    size_t Bytes = InputTgtImage.size();
     performOmptCallback(
         device_load, Plugin.getUserId(DeviceId),
         /*FileName=*/nullptr, /*FileOffset=*/0, /*VmaInFile=*/nullptr,
-        /*ImgSize=*/Bytes, /*HostAddr=*/InputTgtImage->ImageStart,
+        /*ImgSize=*/Bytes,
+        /*HostAddr=*/const_cast<unsigned char *>(InputTgtImage.bytes_begin()),
----------------
jhuber6 wrote:

I'd rather not, it will conflict HORRIBLY with the completely out-of-sync OMPT handling in AOMP.

https://github.com/llvm/llvm-project/pull/158748


More information about the llvm-commits mailing list