<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 10, 2014 at 5:48 AM, Ankur Garg <span dir="ltr"><<a href="mailto:ankur29.garg@samsung.com" target="_blank">ankur29.garg@samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi silvas,<br>
I checked the optimization using ALIVe. I found out a couple of cases, which although never reached this code, were being handled wrong. I have made the appropriate changes to exclude those cases from being handled here as they are already being handled in InstSimplify.<br>
<br>
Since there are 4 number of operators (uge/ugt/ule/ult) and two right shifts (ashr/lshr), the total number of cases are really huge. This makes the output of alive really large. (Total 56 separate cases). Should I paste it here ?<br></blockquote><div><br></div><div><br></div><div>Maybe paste just the input?</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D5518" target="_blank">http://reviews.llvm.org/D5518</a><br>
<br>
Files:<br>
  lib/Transforms/InstCombine/InstCombine.h<br>
  lib/Transforms/InstCombine/InstCombineCompares.cpp<br>
  test/Transforms/InstCombine/icmp-shr.ll<br>
</div></div></blockquote></div><br></div></div>