[PATCH] D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility

Victor Huang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 9 13:51:37 PDT 2021


NeHuang added inline comments.


================
Comment at: clang/lib/Sema/SemaChecking.cpp:3356
+  case PPC::BI__builtin_ppc_maddld:
+    return SemaFeatureCheck(*this, TheCall, "power9-vector",
+                            diag::err_ppc_builtin_only_on_pwr9);
----------------
amyk wrote:
> This is just a question. 
> Is `power9-vector` the correct feature check in these cases? Does it matter if these are not vector instructions?
yeah, we planned using this feature to do the sema check for `pwr9` only (or later cpus) builtins.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102875



More information about the cfe-commits mailing list