[PATCH] D84703: [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary

[Eli Friedman via Phabricator via cfe-commits]

efriedma created this revision.
efriedma added reviewers: sdesmalen, SjoerdMeijer, t.p.northover, cameron.mcinally.
Herald added subscribers: cfe-commits, danielkiss, kristof.beyls.
Herald added a project: clang.

fptosi/fptoui have similar, but not identical, semantics.  In particular, the behavior on overflow is different.

Fixes https://bugs.llvm.org/show_bug.cgi?id=46844 for 64-bit.  (This is missing the corresponding change for 32-bit; the 32-bit equivalent to llvm.aarch64.neon.fcvtzs isn't implemented, as far as I can tell.)


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84703

Files:
  clang/lib/CodeGen/CGBuiltin.cpp
  clang/test/CodeGen/aarch64-neon-fcvt-intrinsics.c
  clang/test/CodeGen/aarch64-neon-intrinsics.c
  clang/test/CodeGen/aarch64-neon-misc.c
  clang/test/CodeGen/aarch64-v8.2a-fp16-intrinsics.c
  clang/test/CodeGen/aarch64-v8.2a-neon-intrinsics.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84703.281052.patch
Type: text/x-patch
Size: 15029 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200727/94e044bb/attachment.bin>