[PATCH] D14805: AMDGPU: Cache information about register pressure sets

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 09:45:11 PST 2015


arsenm added inline comments.

================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.cpp:118-121
@@ -109,1 +117,6 @@
+
+  if (SGPRPressureSets.test(Idx))
+    return SGPRLimit;
+
+  return VGPRLimit;
 }
----------------
Are both SGPRPressureSets and VGPRPressureSets necessary? Can you just have one and assume if it isn't it's the other?


http://reviews.llvm.org/D14805





More information about the llvm-commits mailing list