[PATCH] D29700: [AMDGPU] Implement register pressure callbacks

[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



================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.h:275
+  unsigned getRegPressureLimit(const TargetRegisterClass *RC,
+                               MachineFunction &MF) const override;
+
----------------
rampitec wrote:
> rampitec wrote:
> > arsenm wrote:
> > > const should be added here instead of using const_cast from the other overload?
> > There is no prototype with const MF. Too bad.
> Note, even if I implement my own function taking const MF and call it from non-const overload, the const_cast will just move into that function to call parent's getRegPressureLimit().
I think the parent class should be fixed


Repository:
  rL LLVM

https://reviews.llvm.org/D29700