This is an archive of the discontinued LLVM Phabricator instance.

Fixing -print-module-scope for legacy SCC passes
ClosedPublic

Authored by fedor.sergeev on Nov 21 2018, 6:00 AM.

Details

Summary

It appears that print-module-scope was not implemented for legacy SCC passes.
Fixed to print a whole module instead of just current SCC.

Diff Detail

Repository
rL LLVM

Event Timeline

fedor.sergeev created this revision.Nov 21 2018, 6:00 AM
mkazantsev accepted this revision.Nov 26 2018, 11:48 PM

LGTM

lib/Analysis/CallGraphSCCPass.cpp
644 ↗(On Diff #174910)

Minor: check NeedModule first, it's tad faster.

This revision is now accepted and ready to land.Nov 26 2018, 11:48 PM
This revision was automatically updated to reflect the committed changes.