Page MenuHomePhabricator

[docs] Mention that the legacy PM is deprecated and will be removed after 14
ClosedPublic

Authored by aeubanks on Sep 1 2021, 12:00 PM.

Diff Detail

Event Timeline

aeubanks requested review of this revision.Sep 1 2021, 12:00 PM
aeubanks created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptSep 1 2021, 12:00 PM

This should also be cherry picked into release/13.x

llvm/docs/ReleaseNotes.rst
60

Worth a bit more detail like

"only a minimal level of effort need be made to maintain performance of the legacy pass manager" for the LLVM 14 release as mentioned by @ctetreau , or perhaps phase more briefly.

fhahn added inline comments.Sep 1 2021, 12:14 PM
llvm/docs/ReleaseNotes.rst
60

"only a minimal level of effort need be made to maintain performance of the legacy pass manager" for the LLVM 14 release as mentioned by @ctetreau , or perhaps phase more briefly.

That would be a great addition!

MaskRay accepted this revision.Sep 1 2021, 12:26 PM

LGTM.

This revision is now accepted and ready to land.Sep 1 2021, 12:26 PM
fhahn accepted this revision.Sep 1 2021, 12:41 PM

LGTM, thanks.

llvm/docs/ReleaseNotes.rst
60

I'd go even further and say only minimal effort will be made to maintain the legacy pass manager, not only performance.

fhahn added a comment.Sep 1 2021, 12:43 PM

Also, it might be good to mention the recent llvm-dev thread where this was discussed in the commit message (https://lists.llvm.org/pipermail/llvm-dev/2021-August/152305.html and following)

aeubanks edited the summary of this revision. (Show Details)Sep 1 2021, 1:34 PM
This revision was landed with ongoing or failed builds.Sep 1 2021, 11:32 PM
This revision was automatically updated to reflect the committed changes.

cherry-picked into 13.x