tingwang added a comment.

In D125203#3502433 <https://reviews.llvm.org/D125203#3502433>, @nemanjai wrote:

> Why not also fix this in the front end so that we allow the builtin on the A2 
> CPU as well (since it's supported)?

Oh I missed that. Thank you for pointing out!

Just now updated the patch. However I didn't update the SemaFeatureCheck 
message to indicate the support on a2, since if people see this error message, 
it cannot be a2, and a2 does not easily fit into the message of 
err_ppc_builtin_only_on_arch. Hope this will not create problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125203

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

Reply via email to