This is an archive of the discontinued LLVM Phabricator instance.

[LegacyPM] Remove PlaceSafepointsLegacyPass
ClosedPublic

Authored by kazu on Jun 23 2023, 6:44 PM.

Diff Detail

Event Timeline

kazu created this revision.Jun 23 2023, 6:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 23 2023, 6:44 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
kazu requested review of this revision.Jun 23 2023, 6:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 23 2023, 6:44 PM
nikic accepted this revision.Jun 24 2023, 12:16 AM

LGTM. IIRC this pass is only used out-of-tree, but given that the only user ported it to the NewPM in https://reviews.llvm.org/D136163, I assume it's fine to drop the legacy pass.

This revision is now accepted and ready to land.Jun 24 2023, 12:16 AM
This revision was landed with ongoing or failed builds.Jun 24 2023, 7:34 AM
This revision was automatically updated to reflect the committed changes.