
<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Hi Eric,</div><div><br></div><div>Thanks for the review, I'll add a testcase.</div><div>Regarding the performance impact, tell me what do you want me to measure/run.</div><div><br></div><div>Thanks,</div><div><br></div><div>Quentin<br><br>Envoyé de mon iPhone</div><div><br>Le 11 déc. 2012 à 17:36, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> a écrit :<br><br></div><blockquote type="cite"><div>Seems pretty reasonable, testcase? Performance impact?<div><br></div><div>-eric</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Dec 11, 2012 at 5:22 PM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>LGTM. </div><div><br></div><div><div><div class="h5"><div>On Dec 11, 2012, at 5:20 PM, Quentin Colombet <<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>> wrote:</div>

<br></div></div><blockquote type="cite"><div><div class="h5"><div style="word-wrap:break-word">Hi,<div><br></div><div>I have attached a patch that better controls the inlining of functions when the caller function has MinSize attribute.</div>

<div>Basically, when the caller function has this attribute, we do not "force" the inlining of callee functions carrying the InlineHint attribute (i.e., functions defined with inline keyword).</div><div><br></div>

<div>Thanks for the reviews.</div><div> <br><div>

<div style="font-family:Helvetica;font-size:medium;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;word-wrap:break-word">

-Quentin</div>

</div>

</div></div>

</div></div><span><inliner_minisize.patch></span><div style="word-wrap:break-word"><div></div></div>

_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>

</blockquote></div><br></div><br>_______________________________________________<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" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>

<br></blockquote></div><br></div>

</div></blockquote></body></html>