[PATCH] D14807: AMDGPU: Implement SIRegisterInfo::getRegPressureSetScore()

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 14:30:18 PST 2016


arsenm added inline comments.

================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.cpp:126-127
@@ +125,4 @@
+                                                    unsigned PSetID) const {
+  if (SGPRPressureSets.test(PSetID) && VGPRPressureSets.test(PSetID))
+    return 2;
+
----------------
I'm not sure what this means.

Also you can move the VGPR test under the SGPR pressure set and remove the second if


http://reviews.llvm.org/D14807





More information about the llvm-commits mailing list