[PATCH] D16300: Enable LoopLoadElimination by default

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 27 16:35:36 PST 2016


dberlin added a comment.

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.

(It seems like the time regression is one that we expect to be addressed by the new pass manager, and we understand how to mitigate it if need be)


http://reviews.llvm.org/D16300





More information about the llvm-commits mailing list