[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