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
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
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 re
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: clang/include/c
