[PATCH] D27722: [GVNHoist] Move GVNHoist to function simplification part of pipeline.
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 13 11:38:28 PST 2016
hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.
> Improvements:
Nice! I'm surprised it was not there before; it is not an early-cleanup pass. Putting it right after EarlyCSE should allow it to share a MemorySSA computation with EarlyCSE (at least when we switch it over to using MSSA). LGTM.
https://reviews.llvm.org/D27722
More information about the llvm-commits
mailing list