[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