<div dir="ltr">Justin and Dexon, does this patch look good or do you have any concerns?<div><br></div><div>Thanks,</div><div>Easwaran</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 7, 2015 at 10:31 AM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Easwaran Raman <<a href="mailto:eraman@google.com">eraman@google.com</a>> writes:<br>
> On Fri, Dec 4, 2015 at 4:13 PM, Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br>
</span><span class="">>> Moving this out of the frontend to here makes sense, but I have a couple<br>
>> of questions.<br>
>><br>
>> - Does this only apply when we've supplied actual profile data, or can<br>
>>   we hit it based off of the heuristic counts in no-profile-data mode?<br>
>>   If the latter, this isn't quite NFC.<br>
><br>
> It applies only when there is profile data since entry count and maximum<br>
> function count come only from profile data.<br>
><br>
>> - Can the same be done for the "cold" attribute the frontend also applies?<br>
><br>
> The difference between the inlinehint and cold attributes is that the<br>
> latter is used not just by the inliner. For instance, branch probability<br>
> info. I can move it to the backend and replace all calls<br>
> to hasFnAttr(Attribute::Cold) with a wrapper that also returns true if the<br>
> count is below the threshold. Do you want that change to be in this patch?<br>
<br>
</span>That sounds more invasive, so it probably makes sense to consider it<br>
separately.<br>
</blockquote></div><br></div>