[llvm] [AMDGPU] Enable GCNRewritePartialRegUses pass by default. (PR #72975)

Valery Pykhtin via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 01:37:16 PST 2023


================
@@ -136,13 +136,13 @@ define amdgpu_kernel void @max_256_vgprs_spill_9x32(ptr addrspace(1) %p) #1 {
 ; GFX908-DAG: v_accvgpr_read_b32
 
 ; GFX900: NumVgprs: 256
-; GFX908: NumVgprs: 254
-; GFX900: ScratchSize: 1796
+; GFX908: NumVgprs: 252
+; GFX900: ScratchSize: 132
----------------
vpykhtin wrote:

> Looks like rename-independent-subregs should be shrinking/narrowing registers, but will need a target hook to be able to find if that is possible and which register class to use.
That is what I originally thought to do, but I didn't want to complicate things dealing with other targets. Maybe once this pass get matured it can be made target-independent (it's almost indepent at the moment). I also remember @arsenm had an intention to make it more like an utility class to be used when needed.



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


More information about the llvm-commits mailing list