[PATCH] D25135: [InstCombine] sub X, sext(bool Y) -> add X, zext(bool Y)

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 14:06:25 PDT 2016


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

Please make sure we have a test to check that we can correctly reverse the transform in SelectionDAG when we need to (for example, make sure we don't genereate an unnecessary AND for a vector compare on x86).

Otherwise LGTM.


https://reviews.llvm.org/D25135





More information about the llvm-commits mailing list