[llvm] AMDGPU: Remove let Predicates from PredicateControl'd InstAliases (PR #73474)
Ivan Kosarev via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 27 03:25:14 PST 2023
================
@@ -1069,10 +1067,13 @@ multiclass VOP3P_Real_MFMA_gfx940<bits<7> op, string Name = !cast<VOP3_Pseudo>(N
VOP3Pe_MAI <op, PS_VCD.Pfl, 0>;
} // End AssemblerPredicate = isGFX940Plus, DecoderNamespace = "GFX940"
- defm : VOP3P_Real_MFMA_gfx940_aliases<Name, PS_ACD.Mnemonic, NAME>;
+ let SubtargetPredicate = isGFX940Plus,
+ AssemblerPredicate = isGFX940Plus in {
----------------
kosarev wrote:
That duplicating `AssemblerPredicate` looks like we are trying to work around an overridden `SubtargetPredicate` somewhere. Is this because we do `SubtargetPredicate = HasMAIInsts` above? If so, can we do `OtherPredicates = [HasMAIInsts]` there and remove the `AssemblerPredicate` here?
https://github.com/llvm/llvm-project/pull/73474
More information about the llvm-commits
mailing list