[PATCH] D54283: [DAGCombiner][X86][Mips] Enable combineShuffleOfScalars to run between vector op legalization and DAG legalization. Fix bad one use check in combineShuffleOfScalars

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 8 16:20:43 PST 2018


craig.topper created this revision.
craig.topper added reviewers: RKSimon, spatel.
Herald added subscribers: atanasyan, arichardson, sdardis.

It's possible for vector op legalization to generate a shuffle. If that happens we should give a chance for DAG combine to combine that with a build_vector input.

I also fixed a bug in combineShuffleOfScalars that was considering the number of uses on a undef input to a shuffle. We don't care how many times undef is used.


https://reviews.llvm.org/D54283

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/Mips/cconv/vector.ll
  test/CodeGen/X86/known-signbits-vector.ll
  test/CodeGen/X86/vec_cast.ll
  test/CodeGen/X86/vec_int_to_fp.ll
  test/CodeGen/X86/widen_conv-3.ll
  test/CodeGen/X86/widen_conv-4.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54283.173245.patch
Type: text/x-patch
Size: 22457 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181109/06b3f1d2/attachment.bin>


More information about the llvm-commits mailing list