Page MenuHomePhabricator

[clang] Note that -debug-pass only works with legacy PM when using new PM
AbandonedPublic

Authored by aeubanks on Mar 2 2021, 4:25 PM.

Details

Reviewers
ychen
Summary

Diff Detail

Event Timeline

aeubanks requested review of this revision.Mar 2 2021, 4:25 PM
aeubanks created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptMar 2 2021, 4:25 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
ychen added a comment.Mar 2 2021, 6:08 PM

Sorry, I forgot that codegen is still legacy PM, so -debug-pass=Structure does give the correct output. Then users need to specify both -debug-pass-manager and -debug-pass=x to get the pass information. Hmm, not a big deal but seems hassles. Maybe we should bridge -debug-pass=Structure to -debug-pass-manager ? Or actually, this could be a feature where users can specify either pipeline they want the pass information for. WDYT?

aeubanks abandoned this revision.Mon, Apr 12, 5:02 PM