[PATCH] D44909: [DAGCombine] (float)((int) f) --> ftrunc (PR36617)

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 139871.
spatel added a comment.
Herald added a subscriber: javed.absar.

Patch updated:

1. Add a comment to the DAG node definitions to indicate they have the same behavior as their IR counterparts.
2. Add a basic test file for ARMv7 (let me know if that needs adjusting or more coverage).
3. Added a RUN line for x86 pre-SSE4.1 to ftrunc.ll to show that this patch doesn't change anything for that target.


https://reviews.llvm.org/D44909

Files:
  include/llvm/CodeGen/ISDOpcodes.h
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/ARM/ftrunc.ll
  test/CodeGen/PowerPC/fp-int128-fp-combine.ll
  test/CodeGen/PowerPC/fp-to-int-to-fp.ll
  test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll
  test/CodeGen/X86/2011-10-19-widen_vselect.ll
  test/CodeGen/X86/ftrunc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44909.139871.patch
Type: text/x-patch
Size: 23190 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180327/ea322c1c/attachment.bin>