[llvm] [AMDGPU][MC] Allow null where 128b or larger dst reg is expected (PR #115200)
Jun Wang via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 18 17:56:50 PST 2024
================
@@ -809,6 +809,10 @@ def SReg_32 : SIRegisterClass<"AMDGPU", [i32, f32, i16, f16, bf16, v2i16, v2f16,
let BaseClassOrder = 32;
}
+def SGPR_NULL128 : SIReg<"null">;
----------------
jwanggit86 wrote:
I was actually not very sure if this def is correct. SGPR_NULL64 is defined with 2 sub-regs. I wasn't sure if SGPR_128NULL should be defined similarly, and if so, what the sub-regs should be. On the other hand, changing SGPR_NULL64 to `def SGPR_NULL64: SIReg<"null">` would fail a lot of tests, possibly because SReg_64 is used when it should really be SReg_64_XNULL. Any suggestions are appreciated.
https://github.com/llvm/llvm-project/pull/115200
More information about the llvm-commits
mailing list