[llvm] [AMDGPU] Rework architected SGPRs implementation (PR #79001)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 09:07:26 PST 2024


================
@@ -107,8 +107,8 @@ SIMachineFunctionInfo::SIMachineFunctionInfo(const Function &F,
       MayNeedAGPRs = false; // We will select all MAI with VGPR operands.
   }
 
-  if (!AMDGPU::isGraphics(CC) ||
-      (CC == CallingConv::AMDGPU_CS && ST.hasArchitectedSGPRs())) {
+  if (!AMDGPU::isGraphics(CC) || CC == CallingConv::AMDGPU_CS ||
+      ST.hasArchitectedSGPRs()) {
----------------
jayfoad wrote:

But I need `WorkGroupIDZ` to be set correctly when architected SGPRs are enabled. It is used below, line 177.

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


More information about the llvm-commits mailing list