[PATCH] D100123: [AMDGPU] Add TransVALU to gfx10

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 8 09:23:02 PDT 2021


foad added a comment.

I have no objection to this as a first step in the right direction, but I think it should have no effect at all at the moment. The schedmodel already knows that we can only issue one instruction per cycle, and this patch does not affect that. It only becomes interesting when you model the fact that issuing a trans instruction consumes the trans ALU for (say) 4 cycles, so that on the following 3 cycles you could issue a normal VALU instruction but *not* another trans instruction.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100123



More information about the llvm-commits mailing list