This is an archive of the discontinued LLVM Phabricator instance.

Differential D146243

[docs] Mention removal of legacy optimization pipeline and inliner alloca merging
ClosedPublic

Authored by aeubanks on Mar 16 2023, 10:06 AM.

Details

Reviewers
asbirlea
ctetreau
Commits
rGa63270cda814: [docs] Mention removal of legacy optimization pipeline and inliner alloca…

Diff Detail

Event Timeline

aeubanks created this revision.Mar 16 2023, 10:06 AM
Herald added a reviewer: ctetreau. · View Herald TranscriptMar 16 2023, 10:06 AM
Herald added a project: Restricted Project. · View Herald Transcript
aeubanks requested review of this revision.Mar 16 2023, 10:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 16 2023, 10:06 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
aeubanks mentioned this in D146091: [Inliner] Remove alloca merging.Mar 16 2023, 10:06 AM
asbirlea accepted this revision.Mar 16 2023, 10:37 AM
This revision is now accepted and ready to land.Mar 16 2023, 10:37 AM
This revision was landed with ongoing or failed builds.Mar 16 2023, 10:48 AM
Closed by commit rGa63270cda814: [docs] Mention removal of legacy optimization pipeline and inliner alloca… (authored by aeubanks). · Explain Why
This revision was automatically updated to reflect the committed changes.
aeubanks added a commit: rGa63270cda814: [docs] Mention removal of legacy optimization pipeline and inliner alloca….
sunshaoce added a subscriber: sunshaoce.Mar 16 2023, 11:05 AM
sunshaoce added inline comments.
llvm/docs/ReleaseNotes.rst
65

Here missing a space. :) https://lab.llvm.org/buildbot#builders/30/builds/33006

aeubanks added inline comments.Mar 16 2023, 11:06 AM
llvm/docs/ReleaseNotes.rst
65

my bad, looks like was already fixed in d73c51bfda3719e698ef2e7ed973909d7138d882

Harbormaster completed remote builds in B219908: Diff 505858.Mar 16 2023, 12:02 PM

Revision Contents

PathSize
llvm/
docs/
9 lines

Diff 505875

llvm/docs/ReleaseNotes.rst

Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines
* The ``nofpclass`` attribute was introduced. This allows more * The ``nofpclass`` attribute was introduced. This allows more
optimizations around special floating point value comparisons. optimizations around special floating point value comparisons.
* The constant expression variants of the following instructions have been * The constant expression variants of the following instructions have been
removed: removed:
* ``select`` * ``select``
Changes to LLVM infrastructure
-----------------------------
sunshaoceUnsubmitted
Not Done
ReplyInline Actions

Here missing a space. :) https://lab.llvm.org/buildbot#builders/30/builds/33006

sunshaoce: Here missing a space. :) https://lab.llvm.org/buildbot#builders/30/builds/33006
aeubanksAuthorUnsubmitted
Done
ReplyInline Actions

my bad, looks like was already fixed in d73c51bfda3719e698ef2e7ed973909d7138d882

aeubanks: my bad, looks like was already fixed in d73c51bfda3719e698ef2e7ed973909d7138d882
* The legacy optimization pipeline has been removed.
* Alloca merging in the inliner has been removed, since it only worked with the
legacy inliner pass. Backend stack coloring should handle cases alloca
merging initially set out to handle.
Changes to building LLVM Changes to building LLVM
------------------------ ------------------------
Changes to TableGen Changes to TableGen
------------------- -------------------
Changes to Interprocedural Optimizations Changes to Interprocedural Optimizations
---------------------------------------- ----------------------------------------
▲ Show 20 Lines • Show All 157 Lines • Show Last 20 Lines