[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