[PATCH] D134961: [AMDGPU][MC][GFX11] Correct v_fmac_.*_e64_dpp

Dmitry Preobrazhensky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 04:19:14 PDT 2022


dp added a comment.

>> So for `_e64_dpp` we need both `old` and `src2` and both these operands must be tied to `dst`, correct?
>
> Yes, it seems that would be correct to me.

After a bit of digging I found that we cannot tie more than one operand to another. See https://github.com/llvm/llvm-project/blob/2812a1413f19b9ebd954b57ecbd021656508eade/llvm/utils/TableGen/CodeGenInstruction.cpp#L372
I believe it is the reason why codegen does not support dpp variants of MAC and FMAC opcodes.


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

https://reviews.llvm.org/D134961



More information about the llvm-commits mailing list