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

Ankur Garg ankur29.garg at samsung.com
Mon Oct 6 03:42:31 PDT 2014


Hi David,
All of the logic movable to InstSimplify was already present there. I have removed most of such logic from this function.
Remaining logic is quite specific to certain cases and I have left it as it is, just to avoid confusion in understanding the other cases. Though, it is already being handled in InstSimplify.


Please help in reviewing the updated revision.
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.14444.patch
Type: text/x-patch
Size: 24359 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141006/536b91cb/attachment.bin>


More information about the llvm-commits mailing list