[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-19 Thread Christopher Tetreault via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG55448ab540de: [AArch64] Adding Neon Polynomial vadd Intrinsics (authored by ctetreau). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://revi

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-19 Thread Christopher Tetreault via Phabricator via cfe-commits
ctetreau accepted this revision. ctetreau added a comment. LGTM. I'll land it for you. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://reviews.llvm.org/D96825 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-19 Thread Ryan Santhirarajan via Phabricator via cfe-commits
rsanthir.quic updated this revision to Diff 325034. rsanthir.quic added a comment. rebased on main CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://reviews.llvm.org/D96825 Files: clang/include/clang/Basic/arm_neon.td clang/lib/CodeGen/CGBuiltin.cpp clang/test/Code

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-19 Thread Ryan Santhirarajan via Phabricator via cfe-commits
rsanthir.quic updated this revision to Diff 325027. rsanthir.quic added a comment. Windows builds were failing due to missing builtins in Intrinsics map CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://reviews.llvm.org/D96825 Files: clang/include/clang/Basic/arm_neon.

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-18 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Thanks for the revert! FWIW the failure repros for me locally if I just do `ninja check-clang` (as long as I have the aarch64 target enabled, which it is by default in the cmake build). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-18 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Looks like this breaks check-clang: http://45.33.8.238/win/33493/step_7.txt Please take a look, and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://reviews.llvm.or

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-18 Thread Pengxuan Zheng via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGd9645059c5de: [AArch64] Adding Neon Polynomial vadd Intrinsics (authored by pzheng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://review

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-18 Thread David Spickett via Phabricator via cfe-commits
DavidSpickett accepted this revision. DavidSpickett added a comment. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96825/new/ https://reviews.llvm.org/D96825 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[PATCH] D96825: [AArch64] Adding Neon Polynomial vadd Intrinsics

2021-02-17 Thread Ryan Santhirarajan via Phabricator via cfe-commits
rsanthir.quic updated this revision to Diff 324336. rsanthir.quic marked an inline comment as done. rsanthir.quic retitled this revision from "[AArch64] Adding Polynomial vadd Intrinsics support" to "[AArch64] Adding Neon Polynomial vadd Intrinsics". rsanthir.quic added a comment. removed unneces