[PATCH] D114713: [AArch64][SVE][NEON] Add NEON-SVE-Bridge intrinsics
Peter Waller via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 30 03:37:39 PST 2021
peterwaller-arm added inline comments.
================
Comment at: clang/include/clang/Basic/BuiltinsAArch64NeonSVEBridge.def:2
+#ifdef GET_SVE_BUILTINS
+BUILTIN(__builtin_sve_svget_neonq_s8, "q4iq16bq4i", "n")
+BUILTIN(__builtin_sve_svget_neonq_s16, "q4iq16bq4i", "n")
----------------
Looks like these aren't correct, the number should indicate the number of lanes, and it should be 16 for an s8. `q` indicates scalable, which is also not right for this builtin.
The code which parses these strings can be found here:
https://github.com/llvm/llvm-project/blob/14c4051122bf4070d624b82189f1093758ecdf69/clang/lib/AST/ASTContext.cpp#L10417
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114713/new/
https://reviews.llvm.org/D114713
More information about the cfe-commits
mailing list