[PATCH] D118415: AMDGPU: Reserve v32 if we may need to copy between AGPRs on gfx908

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D118415#3277872 <https://reviews.llvm.org/D118415#3277872>, @rampitec wrote:

> I think it has to be dynamic depending on the requested occupancy. But even then it can drop the occupancy of a kernel if it uses less than 32 registers, which not uncommon. I do not believe we can reserve it that high.

Reserving in the function argument range is a problem. We also treat the requested occupancy as a hint, not something we're forced to follow


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

https://reviews.llvm.org/D118415