[PATCH] D69206: [AMDGPU] Select AGPR in PHI operand legalization

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 21:46:25 PDT 2019


rampitec added a comment.

In D69206#1715472 <https://reviews.llvm.org/D69206#1715472>, @arsenm wrote:

> In D69206#1715469 <https://reviews.llvm.org/D69206#1715469>, @rampitec wrote:
>
> > In D69206#1715466 <https://reviews.llvm.org/D69206#1715466>, @arsenm wrote:
> >
> > > Testcase?
> >
> >
> > Probably I can forge a mir which should not have vgpr inputs for agpr result. So far I cannot figure out why a .mir dumped with -stop-before does not run the next pass (si-fix-sgpr-copies).
>
>
> An IR testcase would also be useful for the copy behavior, it will be useful for RegBankSelect with AGPR eventually


An IR testcase is in the previous patch, mfma-loop.ll. Initialization happens in the entry block, and that demonstrates the problem. I just cannot figure out how to make lit checks.


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

https://reviews.llvm.org/D69206





More information about the llvm-commits mailing list