[PATCH] D130989: [AMDGPU][MC][GFX11] Correct v_dot2_f16_f16 and v_dot2_bf16_bf16

[Jay Foad via Phabricator via llvm-commits]

foad added a subscriber: rampitec.
foad added a comment.

> For v_dot2_bf16_bf16 I am a bit unsure. Because we treat bf16 as int16, will ISel work correctly if we use the FP16InputMods? We don't have a codegen test using modifiers for v_dot2_bf16_bf16, one should probably be added to llvm.amdgcn.fdot2.bf16.bf16.ll . IntOpSelMods is a potential alternative SrcModifiers operand class to use.

+ @rampitec. This already came up on D129084 <https://reviews.llvm.org/D129084>.


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

https://reviews.llvm.org/D130989