<div dir="ltr"><div dir="ltr">Hi All,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 9, 2020 at 12:23 PM Arthur O'Dwyer via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.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"><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" target="_blank">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></div></blockquote><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 class="gmail_quote"><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></div></blockquote><div>Actually there has always been a `-fno-profile-deopt-cold` driver flag in my second Phabricator review (D87338).</div><div>But to sum up, I think it's a good idea to have only one driver flag, or even one LLVM CLI option.</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"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>my $.02,</div><div>–Arthur</div></div></div>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Min-Yih Hsu</div><div>Ph.D Student in ICS Department, University of California, Irvine (UCI).<br></div></div></div></div>