[llvm] r185909 - InstCombine: add more transforms

David Majnemer david.majnemer at gmail.com
Tue Jul 9 02:26:34 PDT 2013


On Tue, Jul 9, 2013 at 2:00 AM, Jay Foad <jay.foad at gmail.com> wrote:

> On 9 July 2013 08:50, David Majnemer <david.majnemer at gmail.com> wrote:
>
>> Author: majnemer
>> Date: Tue Jul  9 02:50:59 2013
>> New Revision: 185909
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=185909&view=rev
>> Log:
>> InstCombine: add more transforms
>>
>
> Thanks!
>
>
>> C1-X <u C2 -> (X|(C2-1)) == C1
>> C1-X >u C2 -> (X|C2) == C1
>> X-C1 <u C2 -> (X & -C2) == C1
>> X-C1 >u C2 -> (X & ~C2) == C1
>>
>
> I assume the second and fourth lines should say != instead of == ? If so,
> the comments in the code have the same bug.
>

Good eye! Thanks, fixed in r185916.


>
> Jay.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130709/d5ed52f3/attachment.html>


More information about the llvm-commits mailing list