[PATCH] D49858: [RFC] re-enable GVNHoist by default

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 27 15:36:54 PDT 2018


sebpop accepted this revision.
sebpop added a comment.
This revision is now accepted and ready to land.

Looks good to me.
Thanks Alexandros for fixing the last known bugs with gvn-hoist.

As Eli mentioned in https://reviews.llvm.org/D48122 the code of gvn-hoist has been used and tested quite extensively in the past year or two.
The initial implementation has been tested with llvm bootstrap, spec 2000, spec 2006, and several mobile proprietary benchmarks.


https://reviews.llvm.org/D49858





More information about the llvm-commits mailing list