[PATCH] D49047: [InstCombine] fix shuffle-of-binops transform to avoid poison/undef

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 8 11:10:59 PDT 2018


nlopes added a comment.

LGTM.

BTW, I couldn't figure if this case is possible, but mentioning it just in case: it's also not legal to introduce 'sdiv undef, ..', since 'sdiv INT_MIN, -1' is UB. Same for srem. So we need to be careful when folding the shuffle on LHS as well.


https://reviews.llvm.org/D49047





More information about the llvm-commits mailing list