Repair a transform that Chris noticed a bug in. Thanks to Nicholas for pointing out my stupid mistakes when writing this patch. :-)

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@45384 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Owen Anderson 2007-12-28 07:42:12 +00:00
parent c48b138869
commit f5783f825d

View File

@ -4835,11 +4835,18 @@ Instruction *InstCombiner::visitICmpInst(ICmpInst &I) {
if (ConstantInt *CI = dyn_cast<ConstantInt>(Op1)) {
Value *A, *B;
#if 0
// (icmp cond (sub A B) 0) -> (icmp cond A B)
if (CI->isNullValue() && match(Op0, m_Sub(m_Value(A), m_Value(B))))
// (icmp cond (sub A B) 0) -> ...
if (CI->isNullValue() && match(Op0, m_Sub(m_Value(A), m_Value(B)))) {
// (icmp cond A B) if cond is signed or equality
if (CmpInst::isSigned(I.getPredicate()) || I.isEquality())
return new ICmpInst(I.getPredicate(), A, B);
#endif
// (icmp ne A B) if cond is ugt
else if (I.getPredicate() == ICmpInst::ICMP_UGT)
return new ICmpInst(ICmpInst::ICMP_NE, A, B);
// (icmp eq A B) if cond is ule
else if (I.getPredicate() == ICmpInst::ICMP_ULE)
return new ICmpInst(ICmpInst::ICMP_EQ, A, B);
}
switch (I.getPredicate()) {
default: break;