This is an archive of the discontinued LLVM Phabricator instance.

[CallPrinter] Port CallPrinter passes to new pass manager
ClosedPublic

Authored by BradSwain on Apr 2 2022, 7:21 PM.

Details

Summary

Port the legacy CallGraphViewer and CallGraphDOTPrinter to work with the new pass manager.

Addresses issue https://github.com/llvm/llvm-project/issues/54323
Adds back related tests that were removed in commits d53a4e7b4a87 and 9e9d9aba1475

Diff Detail

Event Timeline

BradSwain created this revision.Apr 2 2022, 7:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 2 2022, 7:21 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
BradSwain requested review of this revision.Apr 2 2022, 7:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 2 2022, 7:21 PM
aeubanks accepted this revision.Apr 6 2022, 4:05 PM
aeubanks added inline comments.
llvm/lib/Passes/PassRegistry.def
92

alphabetize? (I know not everything is perfect but might as well not make it worse)

This revision is now accepted and ready to land.Apr 6 2022, 4:05 PM
BradSwain updated this revision to Diff 423321.EditedApr 17 2022, 7:03 PM

Thank you for the review! I have alphabetized the new entries in PassRegistry.def.

I don't have commit access, so could someone commit this on my behalf?

Thanks again.

This revision was landed with ongoing or failed builds.Apr 18 2022, 10:02 AM
This revision was automatically updated to reflect the committed changes.