[PATCH] D16300: Enable LoopLoadElimination by default

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 29 11:33:56 PST 2016


anemet added a comment.

In http://reviews.llvm.org/D16300#363677, @dberlin wrote:

> Thanks for doing all this work!
>
> I think this is fine to go in at this point, as long as you are willing to monitor it closely and be ready to disable it if serious regressions are discovered that can't be addressed quickly.


Absolutely and thanks for the review!

I also plan to keep looking at the interaction with GVN Load-PRE, add support for the load-to-load forwarding and see if GVN can be simplified in turn.


http://reviews.llvm.org/D16300





More information about the llvm-commits mailing list