[PATCH] D115526: [InstCombine] don't automatically drop poison-generating flags in SimplifyVectorDemandedElts

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 10 08:36:28 PST 2021


spatel added reviewers: aqjune, hyeongyukim, regehr.
spatel added subscribers: hyeongyukim, aqjune, regehr.
spatel added a comment.

In D115526#3185635 <https://reviews.llvm.org/D115526#3185635>, @nikic wrote:

> It looks like the shufflevector poison change hasn't landed yet? D93818 <https://reviews.llvm.org/D93818> / D103874 <https://reviews.llvm.org/D103874>

Thanks for the reminder! ( cc @aqjune @hyeongyukim @regehr  )

I'm not sure what effect those patches would have here, so maybe we can add test examples to confirm that this doesn't break anything? 
All of the combinations of poison/undef that I tried in Alive2 with shuffles similar to the ones in the diffs here are shown as valid.


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

https://reviews.llvm.org/D115526



More information about the llvm-commits mailing list