<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 19, 2012, at 3:40 AM, Patrik Hägglund H <<a href="mailto:patrik.h.hagglund@ericsson.com">patrik.h.hagglund@ericsson.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="GENERATOR" content="MSHTML 9.00.8112.16450">
<div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial">Can't llvm::PassManagerBuilder::SizeLevel be used?</font></div></div></blockquote></div><div><blockquote type="cite"><div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial"><span class="808063110-19102012"></span></font> </div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial">In my opinion, a function attribute should only be used if there is a need for differencing between functions. For example, if you have a optimization pragma
in the source code, specifying that a particular function (but not other functions) should be optimized (for size in this case).</font></div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial"><span class="808063110-19102012"></span></font> </div></div></blockquote><div>Hi Patrik,</div><div><br></div><div>Good point and in fact, this is what I want to do in my future plan (differencing functions).</div><div>Putting that way, is the patch fine for you?</div><div><br></div><div>Note that with Eli, the current patch is bigger as it includes a test case.</div><div><br></div><div>-Quentin</div><div><br></div><blockquote type="cite"><div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial">Regards,</font></div>
<div dir="ltr" align="left"><font color="#0000ff" size="2" face="Arial">Patrik Hägglund</font><br>
</div>
<div dir="ltr" lang="en-us" class="OutlookMessageHeader" align="left">
<hr tabindex="-1">
<font size="2" face="Tahoma"><b>From:</b> <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:llvm-<a href="mailto:commits-bounces@cs.uiuc.edu">commits-bounces@cs.uiuc.edu</a>]
<b>On Behalf Of </b>Quentin Colombet<br>
<b>Sent:</b> den 18 oktober 2012 20:01<br>
<b>To:</b> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<b>Subject:</b> [llvm-commits] [PATCH] Add an attribute for Oz related optimizations<br>
</font><br>
</div>
<div></div>
<div class="BodyFragment"><font size="2"><span style="FONT-SIZE: 10pt">
<div class="PlainText">Hi,<br>
<br>
The attached patch is the first step to have a better control on Oz related optimizations.<br>
The Oz optimization level focuses on code size, thus I propose to add an attribute called ForceSizeOpt.<br>
<br>
The idea would be to set this attribute to all functions when the Oz optimization level is On (modification not available in this patch) and slowly tune the existing optimizations according to this attribute.<br>
<br>
This is my first patch to llvm, so do not hesitate to make comments please.<br>
<br>
-Quentin<br>
<br>
</div>
</span></font></div>
<div class="BodyFragment"><font size="2"><span style="FONT-SIZE: 10pt">
<div class="PlainText">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div>
</span></font></div>
</div>
</blockquote></div><br></body></html>