<div dir="rtl"><div dir="ltr">r235699 is Pawel's other fix for APInt division. <br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-05-01 3:46 GMT+03:00 Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Which commit is the other fix?<br>
<span class="HOEnZb"><font color="#888888"><br>
-Tom<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Thu, Apr 30, 2015 at 10:13:30PM +0000, Paweł Bylica wrote:<br>
> Hi Tom,<br>
><br>
> It is a fix. However I found another bug in the algorithm. That had been<br>
> fixed also. I think it is good idea to merge both fixes or nothing. I don't<br>
> have any preferences.<br>
><br>
> Paweł<br>
><br>
><br>
> On Thu, Apr 30, 2015, 23:12 Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
><br>
> Ping.<br>
><br>
> It's hard to tell from the commit message, does this fix a bug, or is<br>
> it an optimization?<br>
><br>
><br>
>  Yaron, Chandler, do either of you think this is OK for llvm 3.6?<br>
><br>
> Thanks,<br>
> Tom<br>
><br>
> On Fri, Mar 27, 2015 at 01:39:33PM +0000, Paweł Bylica wrote:<br>
> > Hi Chandler, Yaron & Tom,<br>
> ><br>
> > r233312:<br>
> ><br>
> > > Author: yrnkrn<br>
> > > Date: Thu Mar 26 14:45:19 2015<br>
> > > New Revision: 233312<br>
> > ><br>
> > > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=233312&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=233312&view=rev</a><br>
> > > Log:<br>
> > > Fix rare case where APInt divide algorithm applied un-needed<br>
> > > transformation.<br>
> > ><br>
> > > APInt uses Knuth's D algorithm for long division. In rare cases the<br>
> > > implementation applied a transformation that was not needed.<br>
> > ><br>
> > > Added unit tests for long division. KnuthDiv() procedure is fully<br>
> covered.<br>
> > > There is a case in APInt::divide() that I believe is never used (marked<br>
> > > with<br>
> > > a comment) as all users of divide() handle trivial cases earlier.<br>
> > ><br>
> > > Patch by Pawel Bylica!<br>
> > ><br>
> > >   <a href="http://reviews.llvm.org/D8448" target="_blank">http://reviews.llvm.org/D8448</a><br>
> > ><br>
> > ><br>
> > > Modified:<br>
> > >     llvm/trunk/lib/Support/APInt.cpp<br>
> > >     llvm/trunk/unittests/ADT/APIntTest.cpp<br>
> > ><br>
</div></div></blockquote></div><br></div>