[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