[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 llvm-commits at lists.llvm.org
Mon Sep 20 08:36:18 PDT 2021


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



More information about the llvm-commits mailing list