<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 8, 2018 at 11:16 AM Vedant Kumar via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">vsk added subscribers: tmsriram, tejohnson, rnk.<br>
vsk added a comment.<br>
<br>
Hi Rong, at a high-level, I like what this patch is doing. I'll try to leave in-depth comments by next week -- please ping the review otherwise.<br></blockquote><div><br></div><div>Sounds good. Looking forward to your reviews.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> These suboptimal profile can greatly affect some downstream optimizations, in particular for machine basic block placement optimization.<br>
<br>
If it's possible to get these numbers, it'd be interesting to know how the improvements from this patch compare to link-time or post-link block ordering tools (Bolt).<br></blockquote><div><br></div><div>We mainly use a key google benchmark for performance evaluation. Note that this benchmark has been highly tuned. A performance gain of 0.5% is considered significant. </div><div><br></div><div>Our experiments shows 1.5% to 2% from CSPGO.  The improvement is mainly from machine basicblock placement. CSPGO also sets the accurate function entry count (for the cases that inline are cross module). That will make the hot-text section more dense. This can be seen from instruction heat map. But the performance improvement was small.</div><div><br></div><div>BOLT can have similar performance boosts from  basic block reordering if it's applied to regular PGO binary (i.e. ~2%)</div><div>BOLT can improve the CSPGO binary by 0.5% and that is from function splitting, which is not currently enabled in llvm.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<a href="https://reviews.llvm.org/D54175" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54175</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>