[PATCH] D125203: [PowerPC] Fix PPCISD::STBRX selection issue on A2

Ting Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 9 22:52:52 PDT 2022


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



More information about the llvm-commits mailing list