[PATCH] D16300: Enable LoopLoadElimination by default

[Adam Nemet via llvm-commits]

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?

Thanks.


http://reviews.llvm.org/D16300