This is an archive of the discontinued LLVM Phabricator instance.

[LegacyPM] Remove LowerGuardIntrinsicLegacyPass
ClosedPublic

Authored by kazu on Aug 3 2023, 11:56 PM.

Diff Detail

Event Timeline

kazu created this revision.Aug 3 2023, 11:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2023, 11:56 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
kazu requested review of this revision.Aug 3 2023, 11:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2023, 11:56 PM
kazu added a reviewer: nikic.Aug 3 2023, 11:56 PM
nikic added subscribers: anna, apilipenko.

@anna @apilipenko Do you need this legacy pass (i.e. used in codegen pipeline rather than late opt pipeline)?

anna accepted this revision.Aug 8 2023, 7:27 AM

LGTM - we don't use the legacy pass pipeline. In fact, we're moving away from guard representation completely (not sure if anyone else downstream uses it though).

This revision is now accepted and ready to land.Aug 8 2023, 7:27 AM
This revision was automatically updated to reflect the committed changes.