[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
Tue Sep 30 00:12:16 PDT 2014


Hi David,
Thanks for the comments. Some of the logic that i have written here is already present in InstSimplify. Those cases never reach here. I just added them for better understanding of the code flow as done for previous patch for EQ/NE. Should I remove the logic which is not being used in this function ? or Should I keep it for better understanding of the logical flow ?

Thanks.

http://reviews.llvm.org/D5518






More information about the llvm-commits mailing list