[PATCH] D136527: [InstCombine] use freeze to enable poison-safe logic->select fold

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 22 07:16:34 PDT 2022


spatel added a comment.

In D136527#3876994 <https://reviews.llvm.org/D136527#3876994>, @nikic wrote:

> LGTM. If you're looking for more missing logical and/or folds, I've left quite a few `!IsLogical` breadcrumbs in `foldAndOrOfICmps()`...

Thanks! I was just trying to make sure that simple cases like in the bug report don't escape, so probably don't need to go too much further at the moment, but we'll see... :)


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

https://reviews.llvm.org/D136527



More information about the llvm-commits mailing list