[PATCH] D133967: [IRBuilder] Use PoisonValue in CreateMasked*

[Nuno Lopes via Phabricator via llvm-commits]

nlopes added a comment.

I reviewed all users:

- SLP Vectorizer: the mask is all ones, so passthru is irrelevant
- Loop vectorizer: masked out lanes aren't used. In fact, masked loads are already using poison for passthru.
- X86 expand intrinsic: uses custom zero passthru.
- Codegen expand vp load/gather: there's an issue here as langref says passthru is undef.

So, vector folks, is it ok to change `@llvm.vp.load`/gather's passthru to poison instead of undef?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133967