[clang] [clang][sema][FMV] Forbid multi-versioning arm_streaming functions. (PR #81268)

Sander de Smalen via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 9 08:48:25 PST 2024


================
@@ -3501,9 +3501,18 @@ bool Sema::checkTargetAttr(SourceLocation LiteralLoc, StringRef AttrStr) {
   return false;
 }
 
+static bool isArmStreaming(const FunctionDecl *FD) {
+  if (FD->hasAttr<ArmLocallyStreamingAttr>())
+    return true;
----------------
sdesmalen-arm wrote:

Is this check required? I thought it was more about the interface that can't be both "streaming" and "non-streaming" (for different versions of the function), because the caller needs to know in which mode to call the function. However, the callee is free to change streaming mode in its implementation.

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


More information about the cfe-commits mailing list