[PATCH] D49837: [SelectionDAG] Handle unary SelectPatternFlavor for ABS case in SelectionDAGBuilder::visitSelect.

Ivan Kulagin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 28 13:47:24 PDT 2018


ikulagin updated this revision to Diff 171439.
ikulagin edited the summary of this revision.
ikulagin added a comment.
Herald added subscribers: kristof.beyls, javed.absar.

Rebase and merge changes to one patch.

Well, I have to do changes for NVPTX backend, because the idioms.ll test failed due to the `matchSelectPattern` doesn't recognize the ABS pattern for that case.
(this test uses `icmp sge` but `matchSelectPattern` recognize only `ICMP_SGT` and `ICMP_SLT` cases).

Please pay attention to how I supported Arch and Thumb.
I wait feedback to correct this if it is need.


Repository:
  rL LLVM

https://reviews.llvm.org/D49837

Files:
  include/llvm/CodeGen/TargetLowering.h
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
  lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
  lib/CodeGen/SelectionDAG/LegalizeTypes.h
  lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
  lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp
  lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  lib/CodeGen/SelectionDAG/TargetLowering.cpp
  lib/Target/AArch64/AArch64ISelLowering.cpp
  lib/Target/ARM/ARMISelLowering.cpp
  lib/Target/ARM/ARMISelLowering.h
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/AArch64/iabs.ll
  test/CodeGen/Thumb/iabs.ll
  test/CodeGen/X86/combine-abs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49837.171439.patch
Type: text/x-patch
Size: 27029 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181028/fdd6a82c/attachment.bin>


More information about the llvm-commits mailing list