[clang] [llvm] [FMV][AArch64] Allow user to override version priority. (PR #150267)

Peter Waller via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 8 02:21:46 PST 2025


================
@@ -1627,15 +1661,20 @@ bool SemaARM::checkTargetClonesAttr(
     const StringRef Param = Params[I].trim();
     const SourceLocation &Loc = Locs[I];
 
-    if (Param.empty())
+    auto [LHS, RHS] = Param.split(';');
+    bool HasPriority = !RHS.empty() && RHS.consume_front("priority=");
----------------
peterwaller-arm wrote:

Is it worth trimming whitespace here? If a user wrote a space after the semicolon: `feature; priority=x` the priority could be silently ignored.

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


More information about the llvm-commits mailing list