[all-commits] [llvm/llvm-project] 8dfb5d: [clang codegen][AArch64] Use llvm.aarch64.neon.fcv...

Eli Friedman via All-commits all-commits at lists.llvm.org
Thu Jul 30 15:42:58 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8dfb5d767e70dd862096c8872fd3e3bead99741d
      https://github.com/llvm/llvm-project/commit/8dfb5d767e70dd862096c8872fd3e3bead99741d
  Author: Eli Friedman <efriedma at quicinc.com>
  Date:   2020-07-30 (Thu, 30 Jul 2020)

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

  Log Message:
  -----------
  [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary

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.  (The
corresponding patch for 32-bit is more involved because the equivalent
intrinsics don't exist, as far as I can tell.)

Differential Revision: https://reviews.llvm.org/D84703




More information about the All-commits mailing list