[llvm] [clang] [FMV] Change feature priorities according to ACLE. (PR #79316)

Jon Roelofs via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 09:08:50 PST 2024


================
@@ -659,21 +659,21 @@ int hoo(void) {
 //
 //
 // CHECK: Function Attrs: noinline nounwind optnone
-// CHECK-LABEL: define {{[^@]+}}@fmv_inline._Mfp16fmlMsimd
+// CHECK-LABEL: define {{[^@]+}}@fmv_inline._MsimdMfp16fml
 // CHECK-SAME: () #[[ATTR7]] {
 // CHECK-NEXT:  entry:
 // CHECK-NEXT:    ret i32 14
 //
 //
 // CHECK: Function Attrs: noinline nounwind optnone
-// CHECK-LABEL: define {{[^@]+}}@fmv_inline._Msm4Mfp
----------------
jroelofs wrote:

I think it's okay now since FMV as a feature is in beta and the feature is quite new, but I'm curious what the plan is to manage ABI breaks because of changes like this going forward?

Maybe the features should be sorted lexicographically and not according to their priority.

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


More information about the cfe-commits mailing list