[PATCH] D81661: [VectorCombine] scalarize compares with insertelement operand(s)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D81661#2105403 <https://reviews.llvm.org/D81661#2105403>, @vdmitrie wrote:

> Vector combine dies on assertion with this test case.
>
> ; opt -vector-combine t.ll -S
>
> target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
>  target triple = "x86_64-unknown-linux-gnu"
>
> %struct.s = type { i8* }
>
> ; Function Attrs: nounwind uwtable
>  define <4 x i1> @test(%struct.s** nocapture readonly %t1) #0 {
>
>   %t5 = insertelement <4 x %struct.s**> undef, %struct.s** %t1, i32 0
>   %t6 = icmp ne <4 x %struct.s**> %t5, zeroinitializer
>   ret <4 x i1> %t6
>
> }
>
> attributes #0 = { nounwind uwtable }


Thanks - seems like the assert just became too strict for compares, so I changed it here:
rG741e20f3d61 <https://reviews.llvm.org/rG741e20f3d61964f2d3b1b32971083566be00bab6>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81661