[PATCH] D134723: [AMDGPU] Set memory bound occupancy based on addressable VGPRs

Tue Sep 27 02:12:04 PDT 2022

foad added a comment.

See also D83674 <https://reviews.llvm.org/D83674> (which I more or less gave up on after the reviewer feedback).

In D134723#3817497 <https://reviews.llvm.org/D134723#3817497>, @critson wrote:

> I'm not sure where this is currently tested to add a new test.
> If any reviewers know?

Not sure. `test/CodeGen/AMDGPU/perfhint.ll` tests whether a function is marked as memory bound or not, but I don't know if we have tests for the scheduler allowing memory bound functions to get a lower occupancy.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134723