[llvm] [offload][omp] Move reading _kernel_environment to libomptarget (PR #222606)

Alex Duran via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 07:53:13 PDT 2026


================
@@ -459,8 +465,72 @@ static int loadImagesOntoDevice(DeviceTy &Device) {
               REPORT() << "Failed to write symbol for USM " << Entry.SymbolName;
         } else if (Entry.Address) {
           if (Device.RTL->get_function(Binary, Entry.SymbolName,
-                                       &DeviceEntry.Address) != OFFLOAD_SUCCESS)
+                                       &DeviceEntry.Address) !=
+              OFFLOAD_SUCCESS) {
             REPORT() << "Failed to load kernel " << Entry.SymbolName;
+            continue;
----------------
adurang wrote:

After more careful looking you're right. This would make the Device indexes not be in sync wit the Host table entries and cause problems later. I removed the continue and guarded the rest of the code accordengly.

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


More information about the llvm-commits mailing list