[PATCH] D16300: Enable LoopLoadElimination by default

[Daniel Berlin via llvm-commits]

dberlin added a comment.

FWIW: I agree this should be on by default and will dance with joy when we can remove the current inter-iteration LoadPRE (in favor of a more standard PRE that handles both loads and scalars)

That said, you mention " Long term, the hope is that loop Load-PRE will be removed in favor of this pass at which point those cases will fall on this pass too."

Out of curiosity, have you tried the pass with load pre disabled?
:)


http://reviews.llvm.org/D16300