<div dir="ltr">Hi again,<br><br><div>So the candidates are: <span style="font-size:13.1999998092651px;line-height:19.7999992370605px">r233312 and </span><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">r235699. Both fixes big number division in APInt.</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px"><br></span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">I can perform the merge if accepted.</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px"><br></span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">- Paweł Bylica</span></div><br><div class="gmail_quote">On Fri, May 1, 2015 at 3:30 AM Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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><font color="#888888"><br>
-Tom<br>
</font></span><div><div><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" target="_blank">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>
</blockquote></div></div>