<div dir="ltr">As David had replied OptSize and hasHugeWorkingSetSize have some similarity but still different. For instance, inlining of hot callsites is still aggressive when hasHugeWorkingSetSize is true. If you are suggesting adding a separate function attribute similar to optsize but based on the profile based working set data, I am not sure how useful that is since it is never function specific. </div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 13, 2017 at 5:44 PM, Eli Friedman 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">efriedma added a comment.<br>
<br>
It seems like OptSize and hasHugeWorkingSetSize overlap a lot (a lot of large C++ codebases use -Os to help fit into the icache).  Would it make sense to automatically add OptSize markings based on profile data, or something like that, rather than adding hasHugeWorkingSetSize() checks all over the compiler?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D39992" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D39992</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>