<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 9, 2014 at 12:19 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@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"><br><div><div><div class="h5"><blockquote type="cite"><div>On Sep 9, 2014, at 11:40 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:</div><br><div><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 9, 2014 at 8:57 AM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>----- Original Message -----<br>
> From: "Renato Golin" <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>><br>
> To: "Tilmann Scheller" <<a href="mailto:t.scheller@samsung.com" target="_blank">t.scheller@samsung.com</a>><br>
> Cc: <a href="mailto:reviews%2BD5148%2Bpublic%2B3607c133967fc2d2@reviews.llvm.org" target="_blank">reviews+D5148+public+3607c133967fc2d2@reviews.llvm.org</a>, "LLVM Commits" <<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a>><br>
> Sent: Tuesday, September 9, 2014 9:33:39 AM<br>
> Subject: Re: [PATCH] [AArch64] Enable partial unrolling and runtime unrolling for AArch64 target<br>
><br>
</span><span>> On 9 September 2014 15:15, Tilmann Scheller <<a href="mailto:t.scheller@samsung.com" target="_blank">t.scheller@samsung.com</a>><br>
> wrote:<br>
> > can we make sure this is enabled only with -O3 rather than -O2/-Os<br>
> > though?<br>
> > (at least until the code size regressions are fixed)<br>
><br>
> Good point!<br>
<br>
</span>In case this helps: The way this works is that functions compiled with -Os get the 'optsize' attribute added. The UnrollingPreferences structure has both an PartialThreshold and an PartialOptSizeThreshold parameter that can be set accordingly.<br></blockquote><div><br></div><div>Right, 50% is still a pretty large code size increase even for O2, so O3 might be best here.</div></div></div></div></div></blockquote><div><br></div></div></div>I will go one step further.</div><div><br></div><div>50% code size regression, even on just a few tests, is completely unacceptable for -O2.</div><div><br></div><div>For -O3, it might be tolerable if it provides a very significant performance win to compensate. It doesn’t sound like the performance is really that compelling, though. I think this should be under the control of a separate option, not enabled by default for -O3, until the code size regressions get fixed.</div><span class=""><div><br></div></span></div></blockquote><div><br></div><div>This sounds good to me too.</div><div><br></div><div>-eric <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><span class=""><div></div><div><blockquote type="cite"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Kevin: Are you going to work on the code size issues or...?</div></div></div></div></div></blockquote></div></span></div></blockquote></div><br></div></div>