[PATCH] D70246: [InstCombine] remove identity shuffle simplification for mask with undefs

Zhengyang Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 17 23:09:21 PST 2019


liuz added a comment.

> Converting poison to undef is okay, but undef to poison is not. undef & 0 == undef, but poison & 0 == poison. If undef can be optimized to poison, this can cause miscompilation. See here: https://rise4fun.com/Alive/DFYf , https://rise4fun.com/Alive/OZU

Hi Juneyoung. Is that a typo? undef & 0 should be 0.


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

https://reviews.llvm.org/D70246





More information about the llvm-commits mailing list