[PATCH] D121121: [SLP]Do not schedule instructions with constants/argument/phi operands and external users.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D121121#3387320 <https://reviews.llvm.org/D121121#3387320>, @ronlieb wrote:

> Hi Alexey 
> with this patch, i noticed an assert building one of our runtime files , the test case .c produced is around 24000 lines
> would you like it as is? or reduced ?
>
> Instruction does not dominate all uses!
>
>   %39 = call <8 x i16> @llvm.fshl.v8i16(<8 x i16> %18, <8 x i16> %38, <8 x i16> <i16 8, i16 8, i16 8, i16 8, i16 8, i16 8, i16 8, i16 8>)
>   %23 = shufflevector <8 x i16> %22, <8 x i16> %39, <8 x i32> <i32 0, i32 8, i32 9, i32 10, i32 11, i32 12, i32 13, i32 14>
>
> Instruction does not dominate all uses!
>
>   %67 = call <8 x i16> @llvm.fshl.v8i16(<8 x i16> %44, <8 x i16> %66, <8 x i16> <i16 15, i16 15, i16 15, i16 15, i16 15, i16 15, i16 15, i16 15>)
>   %51 = shufflevector <8 x i16> %50, <8 x i16> %67, <8 x i32> <i32 0, i32 8, i32 9, i32 10, i32 11, i32 12, i32 13, i32 14>
>
> in function eshift
> fatal error: error in backend: Broken function found, compilation aborted!

Hi Ron, it would be good if you can provide reduced case. I'll revert the patch meanwhile.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121121