[PATCH] D159452: [AMDGPU] Machine scheduler should take care of the 'amdgpu-num-vgp' function attribute

[Alexander via Phabricator via llvm-commits]

alex-t added a comment.

Thanks to all who took the time to look at this.
The aim of this review is to discuss and find the right approach.
We have a target description interface that keeps the HW details.
We also have a set of user-defined attributes like: "amdgpu-waves-per-eu", "amdgpu-num-vgpr" etc. which affect the defaults.
We also take into account the LDS size.
Shouldn't we compute the occupancy uniformly accounting for all constraints together?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159452