[PATCH] D121023: [AMDGPU] Add v_mov_b64 gfx940 opcode

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 09:58:56 PST 2022


rampitec added a comment.

In D121023#3363315 <https://reviews.llvm.org/D121023#3363315>, @foad wrote:

> Looks OK to me. I have never really understood why we have duplicate logic for expanding 64-bit vgpr copies in both copyToPhysReg and expandPostRAPseudo.

One is needed for phys regs, the other was mostly created to handle and fold immediate moves (even though works with registers). So the logic isn't really duplicated, copyToPhysReg does not have immediate handling logic.


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

https://reviews.llvm.org/D121023



More information about the llvm-commits mailing list