[PATCH] D141937: [SVE] Add intrinsics for integer binops that explicitly undefine the result for inactive lanes.
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 17 15:49:13 PST 2023
efriedma added a comment.
The dedicated intrinsics seem better than the metadata. The cost of the extra names seems marginal compared to the complexity of ensuring we handle the metadata correctly.
That said, in some of these cases, the intrinsic seems marginally useful. What's the point of inventing llvm.aarch64.sve.add.u if we're going to immediately convert it to an LLVM IR "add"? Why not make clang just emit the "add" directly?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141937/new/
https://reviews.llvm.org/D141937
More information about the llvm-commits
mailing list