[PATCH] D98487: [AArch64][SVE/NEON] Add support for FROUNDEVEN for both NEON and fixed length SVE
Bradley Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 15 03:33:41 PDT 2021
bsmith added a comment.
> Why is this patch only changing int_aarch64_neon_frintn and not int_aarch64_sve_frintn?
> Is there a particular reason to do so?
Things are done slightly differently for SVE in this regard, in principal yes, we could emit roundeven instead of frintn from the ACLE intrinsic, however all of the other ACLE intrinsics also emit SVE specific LLVM intrinsics rather than the arch-indep nodes. This patch doesn't change that in order to stay consistent, if we did want to change that it should be done as a separate patch that changes all of them.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98487/new/
https://reviews.llvm.org/D98487
More information about the cfe-commits
mailing list