[PATCH] Added InstCombine transformation for combining two instructions icmp ult/ule/uge/ugt (ashr/lshr (Const2) %A), (Const1)

Ankur Garg ankur29.garg at samsung.com
Fri Nov 7 06:51:13 PST 2014


Hi everyone,
It seems like this patch which I had submitted earlier for review was quite big and it would've taken a lot of effort for some one to review it.
So, I have split it into 2 parts. 
This updated revision contains the optimization for unsigned operators (ult/ule/ugt/uge). Once this is reviewed, I will submit another patch with only signed operators so, that it becomes easier to review.

Also, Based on another patch i submitted for 'shl' instruction, I have made some changes based on the reviews I received for that patch.

This is a reduced size patch. Please help in reviewing it.

Thanks.

http://reviews.llvm.org/D5518

Files:
  lib/Transforms/InstCombine/InstCombine.h
  lib/Transforms/InstCombine/InstCombineCompares.cpp
  test/Transforms/InstCombine/icmp-shr.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5518.15921.patch
Type: text/x-patch
Size: 12719 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141107/ae27a1f0/attachment.bin>


More information about the llvm-commits mailing list