This is an archive of the discontinued LLVM Phabricator instance.

Register two missing passes with PassRegistry
Needs ReviewPublic

Authored by svenvh on Apr 26 2017, 3:23 AM.

Details

Reviewers
chandlerc
davide
Summary

PrinterLoopPassWrapper and FreeMachineFunction were not registered
with the PassRegistry. Register them for consistency.

Patch by Simone Pellegrini.

Diff Detail

Event Timeline

svenvh created this revision.Apr 26 2017, 3:23 AM
davide requested changes to this revision.Apr 26 2017, 7:24 AM
davide added a subscriber: davide.

I didn't check the pass dependencies but I assumed you did and these passes don't depend on anything.
I think you should be able to verify the change using -run-pass. Please add a test.

This revision now requires changes to proceed.Apr 26 2017, 7:24 AM
svenvh updated this revision to Diff 99927.May 23 2017, 9:10 AM
svenvh edited edge metadata.

Added test, and fixed missing pass dependency.