[llvm] [Offload] Allow setting null arguments in olLaunchKernel and fix amdgpu handling for empty kernel arguments (PR #141958)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 2 07:36:59 PDT 2025


================
@@ -1260,9 +1261,11 @@ struct AMDGPUStreamTy {
     // Consume stream slot and compute dependencies.
     auto [Curr, InputSignal] = consume(OutputSignal);
 
-    // Setup the post action to release the kernel args buffer.
-    if (auto Err = Slots[Curr].schedReleaseBuffer(KernelArgs, MemoryManager))
-      return Err;
+    // Setup the post action to release the kernel args buffer, if it exists
+    if (KernelArgs) {
----------------
jhuber6 wrote:

Fixed it by removing the assert, hopefully it works now. I'm honestly not too concerned with saving the latency of writing a pointer and then doing a no-op.

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


More information about the llvm-commits mailing list