[PATCH] D109301: [AMDGPU] Enable copy between VGPR and AGPR classes during regalloc

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 4 13:00:52 PDT 2021


rampitec added a comment.

In D109301#3109806 <https://reviews.llvm.org/D109301#3109806>, @cdevadas wrote:

> In D109301#3109750 <https://reviews.llvm.org/D109301#3109750>, @rampitec wrote:
>
>> I still do not think we can do it without solving partial spill/copy issue. At least not for wide tuples.
>
> Even today we don't generate partial spills.
> I am not sure how this patch is going to add any incremental impact on existing partial spill behavior.

What do you mean? The code you are removing does it and spill-to-agpr-partial.mir in particular tests it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109301



More information about the llvm-commits mailing list