[PATCH] D137783: [AMDGPU][MC] Support TFE modifiers in MUBUF loads and stores.

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 10 07:36:25 PST 2022


kosarev added a comment.

> AFAIK, `tfe` should be supported for loads and atomics only. I think that there is no sense to use `tfe` with stores.

That was my guess as well. But isn't it right that we still want to be able to disassemble TFE stores, even if functionally useless?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137783



More information about the llvm-commits mailing list