[PATCH] D16300: Enable LoopLoadElimination by default

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 17:23:45 PST 2016

anemet added a comment.

In http://reviews.llvm.org/D16300#330578, @anemet wrote:

> In http://reviews.llvm.org/D16300#330374, @reames wrote:
> > I'd be very tempted to introduce an aggressive and a 
> >  non-aggressive form which differ only in whether they insert runtime 
> >  checks.  I'm concerned by the potential code size and compile time 
> >  impact of aggressive code duplication.
> That's reasonable.  Are you thinking of disabling if SizeLevel is set (i.e. -Os or -Oz) or something different.

This was committed in http://reviews.llvm.org/rL259861.

Is this OK to go in now?



More information about the llvm-commits mailing list