<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 20, 2017 at 12:00 PM, Davide Italiano via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davide added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D32305#732439" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32305#732439</a>, @davidxl wrote:<br>
<br>
> lgtm.<br>
><br>
> by the way,  what is your PM setup for partial inlining?<br>
<br>
<br>
</span>Unfortunately, once I start playing with it, I hit this crash and I started debugging, so I didn't had yet time to put at different places in the pipeline.<br>
You have already tried some configurations? Aside, I noticed partial inlining isn't using BB frequency informations (which I found odd, but probably  it's because nobody cared).<br></blockquote><div><br></div><div>It should use frequency information -- one step a time :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I imagine at least a simple heuristic where you split functions comparing as GCC does, IIRC (or at least, used to do at some point).<br></blockquote><div><br></div><div>Partial inlining will always lead to reduced call overhead with very small impact on size -- so even without using BFI, it should just do fine. My pending patch to improve partial inining will limit the size impact at least.</div><div><br></div><div>David</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There was some initial work on this, but, alas, it's unfinished <a href="https://reviews.llvm.org/D22744" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D22744</a><br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D32305" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32305</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>