<div dir="ltr">Adding Duncan <div><br></div><div>(Something wrong with phabricator, previous mail didn't include Duncan even after adding him as reviewer.)<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>
From: <b class="gmail_sendername">suyog</b> <span dir="ltr"><<a href="mailto:suyog.sarda@samsung.com">suyog.sarda@samsung.com</a>></span><br>Date: Thu, Jul 10, 2014 at 1:55 AM<br>Subject: Re: [PATCH] PR19958 wrong code at -O1 and above on x86_64-linux-gnu (InstCombine)<br>
To: <a href="mailto:suyog.sarda@samsung.com">suyog.sarda@samsung.com</a>, <a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>, <a href="mailto:nlewycky@google.com">nlewycky@google.com</a>, <a href="mailto:spatel@rotateright.com">spatel@rotateright.com</a><br>
Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><br><br>Hi Duncan,<br>
<br>
Thanks for your explanation. I have handled case for 'lshr' separately now in the patch attached.<br>
<br>
I have combined the logic for ashr/lshr as well as exact/non-exact as most of the logic is same.<br>
Wherever there are special cases, i have handled them separately. I have added the logic for the<br>
cases which were handled by 'instsimplify' to keep the logical flow as discussed.<br>
<br>
I have combined the logic, since the 'match' functions are expensive, same for the log,<br>
and our whole point of calculating difference of log was to avoid expensive division operation.<br>
<br>
Please help in reviewing the patch. Your comments/suggestions are most welcomed :)<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D4068" target="_blank">http://reviews.llvm.org/D4068</a><br>
<br>
Files:<br>
  lib/Transforms/InstCombine/InstCombineCompares.cpp<br>
  test/Transforms/InstCombine/icmp.ll<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></div><br><br clear="all"><div><br></div>-- <br>With regards,<br>Suyog Sarda<br>
</div></div>