[PATCH] D107404: [AMDGPU] Avoid assert for saved FP

Fri Aug 6 13:05:38 PDT 2021

rampitec added a comment.

In D107404#2923720 <https://reviews.llvm.org/D107404#2923720>, @rampitec wrote:

> In D107404#2923688 <https://reviews.llvm.org/D107404#2923688>, @arsenm wrote:
>
>> We should stop "spilling" to AGPRs entirely. It's not a spill, it's an ordinary copy due to pressure. We just need to start using the combined VGPR+AGPR classes for values @cdevadas was working on this.
>>
>> The same problem exists for SGPRs and we prune the stack objects before this
>
> All of that is not a reason to assert on a valid program. We also cannot prune stack objects where we create FP copy yet, we do not have spills yet. That is that heurtistic in the `determineCalleeSaves` to produce `WillHaveFP` doing that.

Note, if spilling is replaced with the copying then the option itself will be removed and the assert updated as a result. In the meanwhile we should not assert.

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

https://reviews.llvm.org/D107404