================
@@ -49,16 +49,11 @@ std::optional<AArch64::ArchInfo> 
AArch64::ArchInfo::findBySubArch(StringRef SubA
 }
 
 unsigned AArch64::getFMVPriority(ArrayRef<StringRef> Features) {
----------------
jroelofs wrote:

I'm not sure that this interface can correctly describe the new rule. ISTM it 
would have to be a comparison function between two feature sets, as with this 
implementation, `getFMVPriority({"sme","flagm2","flagm"}) == 
getFMVPriority({"sme","flagm"})` but really we want that to be `>=`.

https://github.com/llvm/llvm-project/pull/121921
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to