[PATCH] D107344: [SLP]Do not emit extra shuffle for insertelements vectorization.

[Steven Johnson via Phabricator via llvm-commits]

srj added a comment.

It's not entirely clear to me how the intended logic of this patch is supposed to work, but skipping the extra shufflevector when `Offset == 0` seems puzzling here; in this case, we are definitely producing a meaningfully different result in this case (essentially, inserting value of 100 at index 3). I'm not sure what the correct fix is here (still looking) but I think this should give you enough info to make progress. (If not, we should should revert this change pending a proper fix.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107344