[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)
More information about the llvm-commits