Index: lib/Transforms/InstCombine/InstCombineAddSub.cpp =================================================================== --- lib/Transforms/InstCombine/InstCombineAddSub.cpp +++ lib/Transforms/InstCombine/InstCombineAddSub.cpp @@ -1047,6 +1047,16 @@ // X + (signbit) --> X ^ signbit if (Val->isSignBit()) return BinaryOperator::CreateXor(LHS, RHS); + + // Is this add the last step in a convoluted sext? + Value *X; + const APInt *C; + if (match(LHS, m_OneUse(m_ZExt(m_OneUse(m_Xor(m_Value(X), m_APInt(C)))))) && + C->isMinSignedValue() && + C->sext(LHS->getType()->getScalarSizeInBits()) == *Val) { + // add(zext(xor i16 X, -32768), -32768) --> sext X + return CastInst::Create(Instruction::SExt, X, LHS->getType()); + } } // FIXME: Use the match above instead of dyn_cast to allow these transforms Index: test/Transforms/InstCombine/apint-add.ll =================================================================== --- test/Transforms/InstCombine/apint-add.ll +++ test/Transforms/InstCombine/apint-add.ll @@ -56,9 +56,7 @@ define i7 @sext(i4 %x) { ; CHECK-LABEL: @sext( -; CHECK-NEXT: [[XOR:%.*]] = xor i4 %x, -8 -; CHECK-NEXT: [[ZEXT:%.*]] = zext i4 [[XOR]] to i7 -; CHECK-NEXT: [[ADD:%.*]] = add nsw i7 [[ZEXT]], -8 +; CHECK-NEXT: [[ADD:%.*]] = sext i4 %x to i7 ; CHECK-NEXT: ret i7 [[ADD]] ; %xor = xor i4 %x, -8 @@ -69,9 +67,7 @@ define <2 x i10> @sext_vec(<2 x i3> %x) { ; CHECK-LABEL: @sext_vec( -; CHECK-NEXT: [[XOR:%.*]] = xor <2 x i3> %x, -; CHECK-NEXT: [[ZEXT:%.*]] = zext <2 x i3> [[XOR]] to <2 x i10> -; CHECK-NEXT: [[ADD:%.*]] = add nsw <2 x i10> [[ZEXT]], +; CHECK-NEXT: [[ADD:%.*]] = sext <2 x i3> %x to <2 x i10> ; CHECK-NEXT: ret <2 x i10> [[ADD]] ; %xor = xor <2 x i3> %x,