[PATCH] D110034: [InstCombine] Update vector transforms to use poison instead of undef for insertelement and shufflevector's placeholder

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

We have a one-vector-input create function in IRBuilder, so it was easy to update that in one place with D93793 <https://reviews.llvm.org/D93793>. Could we do the same for the raw ShuffleVectorInst constructor?

Do we need to update all of these in one commit to avoid regressions, or can this be split up to reduce risk?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110034