This is an archive of the discontinued LLVM Phabricator instance.

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!

wording

Harbormaster completed remote builds in B122141: Diff 370020.Sep 1 2021, 12:15 PM

LGTM.

This revision is now accepted and ready to land.Sep 1 2021, 12:26 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.

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)

wording

aeubanks edited the summary of this revision. (Show Details)Sep 1 2021, 1:34 PM

Harbormaster completed remote builds in B122153: Diff 370043.Sep 1 2021, 1:35 PM

This revision was landed with ongoing or failed builds.Sep 1 2021, 11:32 PM

Closed by commit rG2413d6063b78: [docs] Mention that the legacy PM is deprecated and will be removed after 14 (authored by aeubanks). · Explain Why

This revision was automatically updated to reflect the committed changes.

aeubanks added a commit: rG2413d6063b78: [docs] Mention that the legacy PM is deprecated and will be removed after 14.

cherry-picked into 13.x

Revision Contents

Path

Size

llvm/

docs/

ReleaseNotes.rst

4 lines

Diff 370172

llvm/docs/ReleaseNotes.rst

Show First 20 Lines • Show All 50 Lines • ▼ Show 20 Lines	.. NOTE

Makes programs 10x faster by doing Special New Thing.		Makes programs 10x faster by doing Special New Thing.

* ...		* ...

Changes to the LLVM IR		Changes to the LLVM IR
----------------------		----------------------

* ...		* Using the legacy pass manager for the optimization pipeline is deprecated and
		will be removed after LLVM 14. In the meantime, only minimal effort will be
		MaskRayUnsubmitted Not Done Reply Inline Actions 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. MaskRay: Worth a bit more detail like "only a minimal level of effort need be made to maintain…
		fhahnUnsubmitted Not Done Reply Inline Actions "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! fhahn: > "only a minimal level of effort need be made to maintain performance of the legacy pass…
		fhahnUnsubmitted Not Done Reply Inline Actions I'd go even further and say only minimal effort will be made to maintain the legacy pass manager, not only performance. fhahn: I'd go even further and say only minimal effort will be made to maintain the legacy pass…
		made to maintain the legacy pass manager for the optimization pipeline.

Changes to building LLVM		Changes to building LLVM
------------------------		------------------------

* ...		* ...

Changes to TableGen		Changes to TableGen
-------------------		-------------------
▲ Show 20 Lines • Show All 103 Lines • Show Last 20 Lines

This is an archive of the discontinued LLVM Phabricator instance.

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

Details

Diff Detail

Event Timeline

Revision Contents

Diff 370172

llvm/docs/ReleaseNotes.rst

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