rsandifo-arm added a comment.

LGTM in terms of intent.



================
Comment at: clang/lib/Sema/SemaDecl.cpp:12092
+               diag::err_sme_definition_using_sm_in_non_sme_target);
+        if (UsesZA)
+          Diag(NewFD->getLocation(),
----------------
Suggest making this an `else if`.  We only need to give one error for a call to 
a streaming shared-ZA function, since the underlying error is the same.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157269/new/

https://reviews.llvm.org/D157269

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to