This is an archive of the discontinued LLVM Phabricator instance.

Make sure function passes are run
ClosedPublic

Authored by JesperAntonsson on May 27 2016, 2:57 AM.

Details

Summary

This is a correction to bug 27509.

The reason for the fix is that optlevel flags might have been be cleared, apparently in order to not rerun the AddOptimizationPasses() function. However, this makes us not run FPasses, which should be wrong.

Diff Detail

Event Timeline

JesperAntonsson retitled this revision from to Make sure function passes are run.
JesperAntonsson updated this object.
JesperAntonsson added a reviewer: chandlerc.
JesperAntonsson added a subscriber: llvm-commits.
patrik.h.hagglund edited edge metadata.

Using FPasses exactly when it is initialized (using the replaced condition at line 499) looks like the right thing to do. LGTM.

This revision is now accepted and ready to land.Jun 13 2016, 7:54 AM