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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
