[PATCH] D48223: Allow binop C1, (select cc, CF, CT) -> select folding

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 18 12:27:13 PDT 2018


rampitec updated this revision to Diff 151763.
rampitec marked 5 inline comments as done.
rampitec retitled this revision from "DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"" to "Allow binop C1, (select cc, CF, CT) -> select folding".
rampitec edited the summary of this revision.
rampitec added a comment.
Herald added a subscriber: nemanjai.

- Fixed handling of non-commutative operations if arguments are swapped.
- Added tests for non-commutative operations with all-const value.
- Retitled patch accordingly.


https://reviews.llvm.org/D48223

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/AMDGPU/dagcombine-select.ll
  test/CodeGen/AMDGPU/udivrem.ll
  test/CodeGen/PowerPC/select_const.ll
  test/CodeGen/X86/dagcombine-select.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48223.151763.patch
Type: text/x-patch
Size: 35283 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180618/848154c7/attachment.bin>


More information about the llvm-commits mailing list