[PATCH] D68359: [LegalizeTypes][X86] When splitting a vselect for type legalization, don't split a setcc condition if the setcc input is legal and vXi1 conditions are supported

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 2 15:48:18 PDT 2019


craig.topper created this revision.
craig.topper added reviewers: RKSimon, spatel, efriedma.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

The VSELECT splitting code tries to split a setcc input as well. But on avx512 where mask registers are well supported it should be better to just split the mask and use a single compare.


https://reviews.llvm.org/D68359

Files:
  llvm/lib/CodeGen/SelectionDAG/LegalizeTypesGeneric.cpp
  llvm/test/CodeGen/X86/avx512-vselect.ll
  llvm/test/CodeGen/X86/min-legal-vector-width.ll
  llvm/test/CodeGen/X86/pr34177.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68359.222922.patch
Type: text/x-patch
Size: 12756 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191002/21efc803/attachment.bin>


More information about the llvm-commits mailing list