This is an archive of the discontinued LLVM Phabricator instance.

Move GVNHoist to the right position in the new pass manager pipeline
ClosedPublic

Authored by davide on Jun 1 2017, 3:17 PM.

Details

Summary

GVNHoist was moved as part of simplification passes for the current pass manager (but not for the new), so they're now out-of-sync.

Diff Detail

Event Timeline

davide created this revision.Jun 1 2017, 3:17 PM
chandlerc accepted this revision.Jun 1 2017, 3:21 PM

LGTM, good catch!

This revision is now accepted and ready to land.Jun 1 2017, 3:21 PM
This revision was automatically updated to reflect the committed changes.