[PATCH] D74335: [ARM,MVE] Add intrinsics vclzq and vclsq.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 11 03:35:05 PST 2020


simon_tatham updated this revision to Diff 243792.
simon_tatham added a comment.

Spotted a mistake by myself: I had accidentally implemented the `vclsq` intrinsics for unsigned integer types as well as signed. ACLE only defines them for signed integers.

Also rebased to current master.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74335/new/

https://reviews.llvm.org/D74335

Files:
  clang/include/clang/Basic/arm_mve.td
  clang/include/clang/Basic/arm_mve_defs.td
  clang/test/CodeGen/arm-mve-intrinsics/vclz.c
  llvm/include/llvm/IR/IntrinsicsARM.td
  llvm/lib/Target/ARM/ARMInstrMVE.td
  llvm/test/CodeGen/Thumb2/mve-intrinsics/vcls.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74335.243792.patch
Type: text/x-patch
Size: 8039 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200211/fb321239/attachment.bin>


More information about the llvm-commits mailing list