<div dir="ltr"><div dir="ltr">On Wed, Sep 9, 2020 at 3:10 PM Renato Golin via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Wed, 9 Sep 2020 at 19:26, Nemanja Ivanovic <<a href="mailto:nemanja.i.ibm@gmail.com" target="_blank">nemanja.i.ibm@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">- Default (i.e. no <span style="font-family:monospace">-profile-deopt-cold</span>): do nothing<br></div><div class="gmail_quote"><div>- Option with no arg (i.e. <span style="font-family:monospace">-profile-deopt-cold</span>): add attribute only to functions that have an execution count of zero</div><div>- Option with an arg (i.e. <span style="font-family:monospace">-profile-deopt-cold=<N></span>): add attribute to functions that account for <span style="font-family:monospace"><N>%</span> of total execution counts</div></div></div></blockquote><div><br></div><div>I see. This looks confusing to me, but perhaps it's just me. </div></div></div></blockquote><div><br></div><div>It's not just you. :)  Assuming "account for <N>% of total execution counts" means "account for <N>% <i><b>or less</b></i> of total execution counts," then it seems like the proposed <font face="monospace">-profile-deopt-cold</font> does the same thing as <font face="monospace">-profile-deopt-cold=0</font>.</div><div><br></div><div>Also, for build-system-friendliness, IMHO every positive option should have a negative option — i.e., the default behavior should be regainable via an option such as <font face="monospace">-profile-no-deopt-cold</font>.  (Or <font face="monospace">-fno-profile-deopt-cold</font>, if there was a missing `f` in all of the above.)  That seems easier to do if the whole thing is controlled by just one option instead of two.</div><div><br></div><div>my $.02,</div><div>–Arthur</div></div></div>