[llvm] [AMDGPU][MC]FMA-mix instructions should not be supported in GFX908 etc. (PR #109873)

Stanislav Mekhanoshin via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 27 11:17:23 PDT 2024


================
@@ -1426,7 +1425,6 @@ def FeatureISAVersion9_4_Common : FeatureSet<
   [FeatureGFX9,
    FeatureGFX90AInsts,
    FeatureGFX940Insts,
-   FeatureFmaMixInsts,
----------------
rampitec wrote:

> > I am not saying they are wrong. I am saying it was wrong decision to change instructions and keep the name.
> 
> Could you please elaborate on this? (1) Is the "wrong decision" in the code or in the ISA doc?

In the ISA doc.

> (2) By "change instructions and keep the name", did you mean the mad_mix name is kept in the ISA doc but is actually used for a different instruction than the mad_mix instruction in previous versions, say gfx900?

Yes. And the selection code uses ISD::FMAD to match mad_mix while ISD::FMA for fma_mix.



https://github.com/llvm/llvm-project/pull/109873


More information about the llvm-commits mailing list