[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