[llvm-commits] [llvm] r130405 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp test/Transforms/InstCombine/merge-icmp.ll

Benjamin Kramer benny.kra at googlemail.com
Thu Apr 28 13:15:16 PDT 2011


On 28.04.2011, at 22:03, Duncan Sands wrote:

> Hi Benjamin,
> 
>> --- llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp (original)
>> +++ llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp Thu Apr 28 11:58:40 2011
>> @@ -769,6 +769,42 @@
>>        return Builder->CreateICmp(LHSCC, NewOr, LHSCst);
>>      }
>>    }
>> +
>> +  // (trunc x) == C1&  (and x, CA) == C2 ->  (and CA|CMAX) == C1|C2
> 
> I guess this should be: (and x, CA|CMAX) == C1|C2

Yes, of course. Fixed in r130428.



More information about the llvm-commits mailing list