[PATCH] D79711: [ARM] Add poly64_t on AArch32.

2020-06-05 Thread Ties Stuij via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG1e447318339a: [ARM] Add poly64_t on AArch32. (authored by stuij). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79711/new/ https://reviews.llvm.org/D79711

[PATCH] D79711: [ARM] Add poly64_t on AArch32.

2020-05-28 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard accepted this revision. ostannard added a comment. This revision is now accepted and ready to land. Ok, LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79711/new/ https://reviews.llvm.org/D79711

[PATCH] D79711: [ARM] Add poly64_t on AArch32.

2020-05-26 Thread Alexandros Lamprineas via Phabricator via cfe-commits
labrinea added a comment. > Should poly128_t be available on AArch32 too? I don't see anything in the > ACLE version you linked restricting it to AArch64 only, and the intrinsics > reference has a number of intrinsics available for both ISAs using it. It should but it is not that simple. The

[PATCH] D79711: [ARM] Add poly64_t on AArch32.

2020-05-26 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard added a comment. Should `poly128_t` be available on AArch32 too? I don't see anything in the ACLE version you linked restricting it to AArch64 only, and the intrinsics reference has a number of intrinsics available for both ISAs using it. Comment at: