[PATCH] [AArch64] Enable partial unrolling and runtime unrolling for AArch64 target

[Eric Christopher]

On Tue, Sep 9, 2014 at 8:57 AM, Hal Finkel <hfinkel at anl.gov> wrote:

> ----- Original Message -----
> > From: "Renato Golin" <renato.golin at linaro.org>
> > To: "Tilmann Scheller" <t.scheller at samsung.com>
> > Cc: reviews+D5148+public+3607c133967fc2d2 at reviews.llvm.org, "LLVM
> Commits" <llvm-commits at cs.uiuc.edu>
> > Sent: Tuesday, September 9, 2014 9:33:39 AM
> > Subject: Re: [PATCH] [AArch64] Enable partial unrolling and runtime
> unrolling for AArch64 target
> >
> > On 9 September 2014 15:15, Tilmann Scheller <t.scheller at samsung.com>
> > wrote:
> > > can we make sure this is enabled only with -O3 rather than -O2/-Os
> > > though?
> > > (at least until the code size regressions are fixed)
> >
> > Good point!
>
> 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.
>

Right, 50% is still a pretty large code size increase even for O2, so O3
might be best here.

Kevin: Are you going to work on the code size issues or...?

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140909/3a4da4b0/attachment.html>