[clang] [AArch64][SME] Add diagnostics to CheckConstexprFunctionDefinition (PR #121777)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 6 10:35:22 PST 2025


================
@@ -1854,6 +1855,9 @@ bool Sema::CheckConstexprFunctionDefinition(const FunctionDecl *NewFD,
     }
   }
 
+  if (Context.getTargetInfo().getTriple().isAArch64())
+    ARM().CheckSMEFunctionDefAttributes(NewFD);
+
----------------
cor3ntin wrote:

This seems like a weird place to do that (afaict this has nothing to do with constexpr) and I am not sure this code is tested. I suspect `ActOnStartOfLambdaDefinition` - after the `ProcessDeclAttributes` call might be a better place 

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


More information about the cfe-commits mailing list