[PATCH] D22106: [X86][SSE] Reimplement SSE fp2si conversion intrinsics instead of using generic IR

[Simon Pilgrim via llvm-commits]

RKSimon created this revision.
RKSimon added reviewers: eli.friedman, mkuper, craig.topper, spatel, andreadb.
RKSimon added a subscriber: llvm-commits.
RKSimon set the repository for this revision to rL LLVM.

D20859 and D20860 attempted to replace the SSE (V)CVTTPS2DQ and VCVTTPD2DQ truncating conversions with generic IR instead. 

It turns out that the behaviour of these intrinsics is different enough from generic IR that this will cause problems, INF/NAN/out of range values are guaranteed to result in a 0x80000000 value - which plays havoc with constant folding which converts them to either zero or UNDEF. This is also an issue with the scalar implementations (which were already generic IR and what I was trying to match).

This patch changes both scalar and packed versions back to using x86-specific builtins.

It also deals with the other scalar conversion cases that are runtime rounding mode dependent and can have similar issues with constant folding.

A companion clang patch is at D22105

Repository:
  rL LLVM

http://reviews.llvm.org/D22106

Files:
  include/llvm/IR/IntrinsicsX86.td
  lib/Analysis/ConstantFolding.cpp
  lib/IR/AutoUpgrade.cpp
  lib/Target/X86/X86InstrSSE.td
  test/CodeGen/X86/avx-intrinsics-fast-isel.ll
  test/CodeGen/X86/avx-intrinsics-x86-upgrade.ll
  test/CodeGen/X86/avx-intrinsics-x86.ll
  test/CodeGen/X86/sse-intrinsics-fast-isel-x86_64.ll
  test/CodeGen/X86/sse-intrinsics-fast-isel.ll
  test/CodeGen/X86/sse2-intrinsics-fast-isel-x86_64.ll
  test/CodeGen/X86/sse2-intrinsics-fast-isel.ll
  test/CodeGen/X86/sse2-intrinsics-x86-upgrade.ll
  test/CodeGen/X86/sse2-intrinsics-x86.ll
  test/Transforms/ConstProp/calls.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22106.63110.patch
Type: text/x-patch
Size: 26639 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160707/65ba8b07/attachment.bin>