[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