<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 28, 2016 at 5:54 PM, Piotr Padlewski <span dir="ltr"><<a href="mailto:piotr.padlewski@gmail.com" target="_blank">piotr.padlewski@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Prazek added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D24976#556227" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24976#556227</a>, @tejohnson wrote:<br>
<br>
> In <a href="https://reviews.llvm.org/D24976#556151" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24976#556151</a>, @Prazek wrote:<br>
><br>
> > I have some initial results. It looks like because inliner doesn't know how to inline hot callsites, then most of the SPEC benchmarks have the same binary as current trunk (with hot heuristic).<br>
> >  But on the google branch with smarter inliner there is clear win in povray, about +0.5% compared to current trunk with hot heuristic, over all other benchmarks that produced different binary are not much different.<br>
> ><br>
</span><span class="">> > So the question is - should I temporary set the hot-evolution-factor to the same as normal evolution factor? So this way We won't import more functions that inliner won't inline, and then when new inliner will make it to trunk, to switch it to 1.0<br>
><br>
><br>
</span><span class="">> Can you enable your importing stats to see how many extra funcs we import in trunk?<br>
<br>
<br>
</span>These are the numbers of imported functions for povray<br>
trunk_without_hot      trunk   trunk_hot_chaians<br>
1738                   1863     1876<br>
<br>
So we import 13 functions more. This is kinda expected, there should not be many hot chains<br></blockquote><div><br></div><div>That's fine for povray - but do you have the data for the other benchmarks? Wondering in general how many more funcs are imported across the benchmarks.</div><div><br></div><div>Phab is down, but I downloaded the latest patch: LGTM</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D24976" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24976</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>