[PATCH] D14805: AMDGPU: Cache information about register pressure sets
Tom Stellard via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 22 13:21:07 PDT 2016
tstellarAMD added inline comments.
================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.cpp:118-121
@@ -109,1 +117,6 @@
+
+ if (SGPRPressureSets.test(Idx))
+ return SGPRLimit;
+
+ return VGPRLimit;
}
----------------
arsenm wrote:
> Are both SGPRPressureSets and VGPRPressureSets necessary? Can you just have one and assume if it isn't it's the other?
It's because of the VS_32 register class has both SGPRs. and VGPRs.
http://reviews.llvm.org/D14805
More information about the llvm-commits
mailing list