[PATCH] D137435: [InstSimplify] (~A & B) | ~(A | B) --> ~A with logical and

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 09:25:05 PDT 2022


spatel added a comment.

I wonder if we could replace all of the matchers in this whole function. We can't leak poison when simplifying to an existing value? This will need a pile of tests...

Also, if any logic expression folds to a constant, then a transform must be safe to match with logical and/or sub-expressions too?


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

https://reviews.llvm.org/D137435



More information about the llvm-commits mailing list