================
@@ -211,7 +211,7 @@ def HasSME2p2        : Predicate<"Subtarget->isStreaming() 
&& Subtarget->hasSME2
                                  AssemblerPredicateWithAll<(all_of 
FeatureSME2p2), "sme2p2">;
 def HasSVEAES2       : Predicate<"Subtarget->hasSVEAES2()">,
                                  AssemblerPredicateWithAll<(all_of 
FeatureSVEAES2), "sve-aes2">;
-def HasSVEBFSCALE    : Predicate<"Subtarget->isSVEorStreamingSVEAvailable() && 
Subtarget->hasSVEBFSCALE()">,
+def HasSVEBFSCALE    : Predicate<"Subtarget->isSVEorStreamingSVEAvailable() && 
Subtarget->hasSVE_BFSCALE()">,
----------------
Lukacma wrote:

What you want here for SVE BFSCALE is this:


```suggestion
def HasSVEBFSCALE    : Predicate<"Subtarget->isNonStreamingSVEorSME2Available() 
&& Subtarget->hasSVE_BFSCALE()">,
```

Though usage of this flag in SME BFSCALE is odd (sorry about that) as 
isSVEorStreamingSVEAvailable part is redundant there. So maybe creating new 
predicate and simplifying this one for use in SME BFSCALE would be better.

https://github.com/llvm/llvm-project/pull/172025
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to