This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Remove CodegenCleanupPass.

Authored by fhahn on Feb 18 2023, 12:40 PM.



The pass uses a bunch of deprecated legacy passes and appears unused.
Remove it to unblock removing legacy passes.


Diff Detail

Event Timeline

fhahn created this revision.Feb 18 2023, 12:40 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: StephenFan. · View Herald Transcript
fhahn requested review of this revision.Feb 18 2023, 12:40 PM

Looks like this is dead code since This LGTM, but probably best to give @Meinersbur a chance to comment -- I'm not sure whether these passes not being part of the NewPM pipeline was an oversight?

Meinersbur accepted this revision.Feb 21 2023, 9:50 PM


In the NPM, Polly uses buildFunctionSimplificationPipeline to cleanup after Polly. This was for the legacy pass manager.

This revision is now accepted and ready to land.Feb 21 2023, 9:50 PM
fhahn updated this revision to Diff 500140.Feb 24 2023, 4:17 AM

Thanks, rebase before landing.

This revision was landed with ongoing or failed builds.Feb 24 2023, 4:39 AM
This revision was automatically updated to reflect the committed changes.