This is an archive of the discontinued LLVM Phabricator instance.

[NFC][2/n] Remove PrunePH pass
ClosedPublic

Authored by speryt on Sep 26 2022, 5:36 PM.

Details

Summary

Second patch in the series to remove legacy PM and
associated -enable-new-pm=0 flag targets pass that
has not been ported to new PM - PruneEH.
Discussion about this can be found in D44415.

Diff Detail

Event Timeline

speryt created this revision.Sep 26 2022, 5:36 PM
Herald added a project: Restricted Project. · View Herald Transcript
speryt requested review of this revision.Sep 26 2022, 5:36 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 26 2022, 5:36 PM
speryt added inline comments.Sep 26 2022, 5:37 PM
llvm/test/Transforms/PruneEH/pr26263.ll
6

I have second thought whether all of PrunePH directory tests should be dropped if they have two RUN lines.

aeubanks accepted this revision.Sep 26 2022, 5:40 PM
aeubanks added inline comments.
llvm/test/Transforms/PruneEH/pr26263.ll
6

the functionality in simplifycfg/function-attrs that replace pruneeh should theoretically have their own tests

This revision is now accepted and ready to land.Sep 26 2022, 5:40 PM
This revision was landed with ongoing or failed builds.Sep 26 2022, 6:38 PM
Closed by commit rG46fc75ab28b7: [NFC][2/n] Remove PrunePH pass (authored by speryt, committed by aeubanks). · Explain Why
This revision was automatically updated to reflect the committed changes.
llvm/test/Transforms/PruneEH/seh-nounwind.ll