<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 7, 2015 at 10:53 AM, Diego Novillo <span dir="ltr"><<a href="mailto:dnovillo@google.com" target="_blank">dnovillo@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dnovillo added a comment.<br>
<br>
Once this is in, please update lib/Transforms/IPO/SampleProfile.cpp. It does something similar.<br></blockquote><div>It does something similar, but not the same (hotness is based on ratio of samples in the function to the samples in the program). I'll work on a common interface that will work for the current frontend instrumentation based PGO, the backend instrumentation based  PGO Rong is proposing and sampled PGO.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, should cold functions be marked with the cold hint?<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div>I'll send a separate patch for that as  suggested by Justin and David.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D15245" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15245</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>