[PATCH] D125126: [AMDGPU] Enable FLAT LDS DMA on gfx9/10 before gfx940

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 16:14:24 PDT 2022


rampitec added a comment.

A potentially better alternative is to use gfx940 names with _LDS_ in the mnemonic instead of a modifier. This is logically a different opcode anyway. The only downside it is not compatible with the documentation and sp3. But then it was not implemented before and therefore not used, so there shall be no compatibility problem on practice. Well, it will also be different from MUBUF. Given the difference in both semantics and addressing mode I personally would prefer it to be different opcodes. At a pseudo level it is certainly easier to have separate ops for this.

Preferences?


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

https://reviews.llvm.org/D125126



More information about the llvm-commits mailing list