<div dir="ltr"><div dir="ltr">HI Renato,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 10, 2020 at 10:54 AM Renato Golin <<a href="mailto:rengolin@gmail.com">rengolin@gmail.com</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 Thu, 10 Sep 2020 at 18:22, Min-Yih Hsu <<a href="mailto:minyihh@uci.edu" target="_blank">minyihh@uci.edu</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>I totally agree with Wenlei that (somewhere in the future) we should have a unified FDO framework for both code size and compilation time. And I think Renato and Tobias's suggestions to do the same thing for size-oriented attributes (i.e. `minsize` and `optsize`) is the low-hanging fruit we can support in a short time.</div><div>Engineering-wised I'll prefer to send out a separate review for the size-oriented attributes work, since `minsize` / `optsize` are kind of in conflict with `optnone` so I don't think it's a good idea to put them into one flag / feature set.</div></div></blockquote><div><br></div><div>I'm happy for this unification to happen at a later stage. Just not too long later.</div><div><br></div><div>I worry exposing the flags will get people to use it and then we'll change it. The longer we leave it, the more people will be hit by the subtle change.</div><div><br></div><div>Worse still if we release with one behaviour now and then with a different behaviour in the next release. </div></div></div></blockquote><div><br></div><div>Yeah totally understandable. I will try to not let it happen.</div><div>I'm willing to implement the support for optsize / minsize after this patch. As well as providing some experiment numbers to justify it. </div><div><br></div><div>Best,</div><div>Min</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>Having conditional paths in build systems for different versions of the compiler isn't fun.</div><div><br></div><div>--renato</div></div></div>
</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>