[llvm-branch-commits] [clang] [llvm] [AArch64][llvm] Armv9.8-A: Add support for FEAT_SVE_PERM256 (PR #231433)

[via llvm-branch-commits]

================
@@ -322,6 +324,11 @@ def HasNonStreamingSVE2p2_or_SME2p2
                 AssemblerPredicateWithAll<(any_of FeatureSVE2p2, FeatureSME2p2),
                 "sme2p2 or sve2p2">;
 
+def HasMatMulFP64_or_SVE_PERM256
+    : Predicate<"Subtarget->hasSVE_PERM256() || (Subtarget->isSVEAvailable() && Subtarget->hasMatMulFP64())">,
+                AssemblerPredicateWithAll<(any_of FeatureMatMulFP64, FeatureSVE_PERM256),
+                "f64mm or sve-perm256">;
+
----------------
Lukacma wrote:

I think it would be cleaner to capture the whole condition here. Maybe smth like this:

```suggestion
def HasNonStreamingMatMulFP64_or_SVE_PERM256
    : Predicate<"(Subtarget->isSVEorStreamingSVEAvailable()  && Subtarget->hasSVE_PERM256()) || (Subtarget->isSVEAvailable() && Subtarget->hasMatMulFP64())">,
                AssemblerPredicateWithAll<(any_of FeatureMatMulFP64, FeatureSVE_PERM256),
                "f64mm or sve-perm256">;

```

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