<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 28, 2016 at 9:04 AM, Sanjay Patel <span dir="ltr"><<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">spatel accepted this revision.<br>
spatel added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
In <a href="http://reviews.llvm.org/D21560#468552" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21560#468552</a>, @danielcdh wrote:<br>
<br>
> I just tested the perf impact of this patch for SPECCPU INT2006 benchmarks (perf test were done on sandybridge):<br>
<span class="">><br>
> 400.perlbench 2.21%<br>
>  401.bzip2    -0.46%<br>
>  403.gcc      1.59%<br>
>  429.mcf      1.79%<br>
>  445.gobmk    1.52%<br>
>  456.hmmer    1.63%<br>
>  458.sjeng    5.62%<br>
>  462.libquantum       1.54%<br>
>  464.h264ref  -1.28%<br>
>  471.omnetpp  0.00%<br>
>  473.astar    2.35%<br>
>  483.xalancbmk        -0.27%<br>
>  overall      1.29%<br>
<br>
<br>
</span>That is a surprisingly large gain. This raises the question: if 64 is good, is 128 even better?<br></blockquote><div><br></div><div>Tried with 128, the perf diff does not seems to be out of noise range. Thus let's stick to 64.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><br>
> About the -Oz issue, I agree that there should be no xor inserted at -Oz. But that's a different issue and should be fixed in a separate patch.<br>
<br>
<br>
</span>Agreed. Please let me know if you plan to follow-up on that part.<br>
<br>
I have no further comments on this patch, so LGTM.<br></blockquote><div><br></div><div>Thanks, as the original author (<span style="font-size:12.8px">stoklund)</span> of the number "16" has not responded, and he seems to be inactive since one year ago, I'll go ahead submit the patch.</div><div><br></div><div>Dehao</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D21560" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21560</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>