[llvm] [Offload] Return an integer user count for AMDGPU HSA queues (PR #223335)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 23 23:57:23 PDT 2026
https://github.com/StevenYangCC updated https://github.com/llvm/llvm-project/pull/223335
>From 1fd38586b3cf61054d0a8bd5f370cefe92f2ad74 Mon Sep 17 00:00:00 2001
From: "chengcang.yang" <yangchengcang at gmail.com>
Date: Thu, 24 Sep 2026 14:55:57 +0800
Subject: [PATCH] [Offload] Return an integer user count for AMDGPU HSA queues
getUserCount returned bool while the stored count is uint32_t, so
busy-queue selection treated every non-zero count as one. Return the
integer count so least-used assignment can distinguish differently
loaded queues.
---
offload/plugins-nextgen/amdgpu/src/rtl.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/offload/plugins-nextgen/amdgpu/src/rtl.cpp b/offload/plugins-nextgen/amdgpu/src/rtl.cpp
index 92afb883840ef..8169231a5ceaa 100644
--- a/offload/plugins-nextgen/amdgpu/src/rtl.cpp
+++ b/offload/plugins-nextgen/amdgpu/src/rtl.cpp
@@ -814,7 +814,7 @@ struct AMDGPUQueueTy {
}
/// Returns the number of streams, this queue is currently assigned to.
- bool getUserCount() const { return NumUsers; }
+ uint32_t getUserCount() const { return NumUsers; }
/// Returns if the underlying HSA queue is initialized.
bool isInitialized() { return Queue != nullptr; }
More information about the llvm-commits
mailing list