Index: llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp =================================================================== --- llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp +++ llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp @@ -1425,13 +1425,18 @@ return BinaryOperator::CreateAnd(Op1, Builder->CreateNot(C)); // (A | B) & ((~A) ^ B) -> (A & B) - if (match(Op0, m_Or(m_Value(A), m_Value(B))) && - match(Op1, m_Xor(m_Not(m_Specific(A)), m_Specific(B)))) + // (A | B) & (B ^ (~A)) -> (A & B) + // (B | A) & ((~A) ^ B) -> (A & B) + // (B | A) & (B ^ (~A)) -> (A & B) + if (match(Op1, m_c_Xor(m_Not(m_Value(A)), m_Value(B))) && + match(Op0, m_c_Or(m_Specific(A), m_Specific(B)))) return BinaryOperator::CreateAnd(A, B); // ((~A) ^ B) & (A | B) -> (A & B) // ((~A) ^ B) & (B | A) -> (A & B) - if (match(Op0, m_Xor(m_Not(m_Value(A)), m_Value(B))) && + // (B ^ (~A)) & (A | B) -> (A & B) + // (B ^ (~A)) & (B | A) -> (A & B) + if (match(Op0, m_c_Xor(m_Not(m_Value(A)), m_Value(B))) && match(Op1, m_c_Or(m_Specific(A), m_Specific(B)))) return BinaryOperator::CreateAnd(A, B); } Index: llvm/trunk/test/Transforms/InstCombine/and.ll =================================================================== --- llvm/trunk/test/Transforms/InstCombine/and.ll +++ llvm/trunk/test/Transforms/InstCombine/and.ll @@ -617,10 +617,7 @@ define i32 @test42(i32 %a, i32 %c, i32 %d) { ; CHECK-LABEL: @test42( ; CHECK-NEXT: [[FORCE:%.*]] = mul i32 [[C:%.*]], [[D:%.*]] -; CHECK-NEXT: [[OR:%.*]] = or i32 [[FORCE]], [[A:%.*]] -; CHECK-NEXT: [[NOTA:%.*]] = xor i32 [[A]], -1 -; CHECK-NEXT: [[XOR:%.*]] = xor i32 [[FORCE]], [[NOTA]] -; CHECK-NEXT: [[AND:%.*]] = and i32 [[XOR]], [[OR]] +; CHECK-NEXT: [[AND:%.*]] = and i32 [[FORCE]], [[A:%.*]] ; CHECK-NEXT: ret i32 [[AND]] ; %force = mul i32 %c, %d ; forces the complexity sorting @@ -634,10 +631,7 @@ define i32 @test43(i32 %a, i32 %c, i32 %d) { ; CHECK-LABEL: @test43( ; CHECK-NEXT: [[FORCE:%.*]] = mul i32 [[C:%.*]], [[D:%.*]] -; CHECK-NEXT: [[OR:%.*]] = or i32 [[FORCE]], [[A:%.*]] -; CHECK-NEXT: [[NOTA:%.*]] = xor i32 [[A]], -1 -; CHECK-NEXT: [[XOR:%.*]] = xor i32 [[FORCE]], [[NOTA]] -; CHECK-NEXT: [[AND:%.*]] = and i32 [[OR]], [[XOR]] +; CHECK-NEXT: [[AND:%.*]] = and i32 [[FORCE]], [[A:%.*]] ; CHECK-NEXT: ret i32 [[AND]] ; %force = mul i32 %c, %d ; forces the complexity sorting