[openmp] [clang] [llvm] [OpenMP] Remove `register_requires` global constructor (PR #80460)

Joseph Huber via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 2 13:59:41 PST 2024


================
@@ -199,7 +199,7 @@ static int initLibrary(DeviceTy &Device) {
                                         Entry.size) != OFFLOAD_SUCCESS)
               REPORT("Failed to write symbol for USM %s\n", Entry.name);
           }
-        } else {
+        } else if (Entry.addr) {
----------------
jhuber6 wrote:

Yes, I'll make the offloading entries less dumb in the future. I don't like how for working w/ CUDA and stuff we need `cuda_offloading_entries` and  whatnot as well. I've been planning on merging them and simply having a field for the language kind and stuff. Been planning on doing that for awhile.

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


More information about the cfe-commits mailing list