[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