[PATCH] D81013: [InstCombine] move vector select ahead of select-shuffle

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D81013#2074535 <https://reviews.llvm.org/D81013#2074535>, @srj wrote:

> This seems to have injected breakages into Halide when used for x86-64 targets with 'narrow' SIMD (eg., using only SSE2 or SSE4.1); many of our tests now fail in that configuration with variations of
>
>   /path/tollvm-project/llvm/lib/IR/Instructions.cpp:2049: bool isSingleSourceMaskImpl(llvm::ArrayRef<int>, int): Assertion `Mask[i] >= 0 && Mask[i] < (NumOpElts * 2) && "Out-of-bounds shuffle mask element"' failed.
>


Looks like we are missing `!increasesLength()` check somewhere.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81013/new/

https://reviews.llvm.org/D81013