[PATCH] D29798: [AMDGPU] Override PSet for M0

[Matt Arsenault via Phabricator via llvm-commits]

arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with param name fixed



================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.cpp:1333
+
+const int *SIRegisterInfo::getRegUnitPressureSets(unsigned RegUnit) const {
+  static const int Empty[] = { -1 };
----------------
Ditto


================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.h:214
 
+  const int *getRegUnitPressureSets(unsigned RegUnit) const override;
+
----------------
Param name is misleading, it should be Reg not RegUnit


Repository:
  rL LLVM

https://reviews.llvm.org/D29798