[llvm] [offload] add handling of memory alignment to MemoryManagerTy (PR #218418)

Jan Trusiłło via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 06:19:55 PDT 2026


================
@@ -615,18 +615,6 @@ struct CUDADeviceTy : public GenericDeviceTy {
     if (auto Err = Plugin::check(Res, "error in cuMemAlloc[Host|Managed]: %s"))
       return std::move(Err);
 
-    if (Alignment > 0 && !isAddrAligned(Align(Alignment), MemAlloc)) {
----------------
311Volt wrote:

> The alignment is verified at the memory manager level now on the path forwarding the allocation directly to the device - it is not removed entirely.

`MemoryManager` is not actually mandatory (you can disable it with `LIBOMPTARGET_MEMORY_MANAGER_THRESHOLD=0`), so now if you disable the memory manager you also bypass the alignment check

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


More information about the llvm-commits mailing list