[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 186963. steleman added a comment. Addressed comments from Renato: - Diagnostic for unsupported argument to -fveclib= is now: error: unsupported option '' for target '' - Added check and diagnostic for SVML on non-Intel ISA's. Repository: rC Clang

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman added a comment. Hi Renato, Thank you very much for the comments. I will create a new changeset incorporating your comments, and then re-submit. I will also add a Diagnostic for SVML and X86/X86_64. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53928/n

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 185578. steleman added a comment. Herald added a project: clang. This is a small - but signifcant - update to the original changeset. What's new in this update: ${top_srcdir}/lib/Frontend/CompilerInvocation.cpp: When -fveclib=SLEEF was passed on compile-li

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman added a comment. Ping! Yes, I know, everyone was away for the holidays. :-) Could someone please take a look at this. We'd like to make sure everything is OK on this side before D53927 can proceed. Yes, it's a bit of a circular dependency in that sens

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 178884. steleman added a comment. Removed spurious patch for an unrelated change. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53928/new/ https://reviews.llvm.org/D53928 Files: include/clang/Basic/Builtins.def include/

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 178477. steleman added a comment. Herald added a subscriber: arphaman. Updated version of this changeset/patch, as per Renato's latest comments from D53927 (https://reviews.llvm.org/D53927). New benchmark comparing SLEEF pe

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 172638. steleman added a comment. - changed the -fveclib= argument value to 'sleefgnuabi'. - added atan2 and pow. - spreadsheet with comparison between libm and sleef is here: https://docs.google.com/spreadsheets/d/1lcpESCnuzEoTl_XHBqE9FLL0tXJB_tZGR8yciCx1yj

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman created this revision. steleman added a reviewer: joel_k_jones. Herald added subscribers: kristina, kristof.beyls, javed.absar. This changeset adds the required Builtins for enabling AArch64 vectorization of libm trigonometry functions via SLEEF: http://sleef.org/. - A new argument is a