[PATCH] D117844: [AMDGPU] Prevent aliasing of SrcC and Dst in MAI

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 13:44:19 PST 2022


rampitec planned changes to this revision.
rampitec added a comment.

In D117844#3261814 <https://reviews.llvm.org/D117844#3261814>, @rampitec wrote:

> In D117844#3261783 <https://reviews.llvm.org/D117844#3261783>, @foad wrote:
>
>> I would suggest instruction selecting to the version with tied operands, and then letting TwoAddressInstruction decide whether to call your convertToThreeAddress, which would convert it to the earlyclobber form.
>
> So that would mean 4 pseudos for each instruction? 2 different versions for accvgpr vs vccvgpr by 2 variants with tied operands vs earlyclobber?
> I cannot really do this trick with tied operand at/after selection, it got malformed after operand folding.

OK, that seems to be working in principle.


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

https://reviews.llvm.org/D117844



More information about the llvm-commits mailing list