[PATCH] D76417: [AMDGPU] Set a cost model for vgpr registers.

[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 except don't call it a cost model in the final commit message, it's just the cost



================
Comment at: llvm/lib/Target/AMDGPU/SIRegisterInfo.td:220
 foreach Index = 0-255 in {
+  // set a cost value for vgprs other than the argument registers (v0-v31).
+  // The ratio of index/allocation_granularity is taken as the cost value.
----------------
Capitalize


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76417/new/

https://reviews.llvm.org/D76417