This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Remove old CI configurations and update the supported compiler versions
ClosedPublic

Authored by philnik on Sep 6 2022, 10:03 AM.

Details

Summary

_LIBCPP_REMOVE_TRANSITIVE_INCLUDES doesn't do anything anymore in C++23 mode, so it's now just a duplicate of the C++23 configuration.
Also add new steps to the post-release checklist for updating the supported compilers.

Diff Detail

Event Timeline

philnik created this revision.Sep 6 2022, 10:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 6 2022, 10:03 AM
philnik requested review of this revision.Sep 6 2022, 10:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 6 2022, 10:03 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne requested changes to this revision.Sep 7 2022, 7:17 PM

Please coordinate with @Mordante. I just approved a patch that removed support for Clang 13. I think this could become only a removal of the generic-no-transitive-includes CI configuration.

This revision now requires changes to proceed.Sep 7 2022, 7:17 PM

Thanks for working on this!

libcxx/docs/index.rst
107

I would prefer to only mention releases here. Having Clang 16 may lead to confusion since it's not released yet.
For AppleClang we already have some version 14 support.
I feel having support for the work-in-progress function can be left implicit.

libcxx/utils/ci/buildkite-pipeline.yml
259

This part has already landed.

505

I didn't remove this part, and I would prefer it in a separate commit; which can be this commit.

philnik added inline comments.Sep 12 2022, 2:05 AM
libcxx/docs/index.rst
107

I was actually always confused by the latest two stable releases. The released versions also only claim support for the last two versions, i.e. libc++14 claims compatibility with clang 12 and 13, but not 14 (see https://releases.llvm.org/14.0.0/projects/libcxx/docs/index.html), which I find very confusing. The new wording makes the situation clear.

libcxx/utils/ci/buildkite-pipeline.yml
505

I'll split it if we want to reword the supported compilers.

Mordante added inline comments.Sep 12 2022, 10:19 AM
libcxx/docs/index.rst
107

I see what you mean. However I think this wording doesn't work. When LLVM 16 will be released and current trunk would refer to LLVM 17. So that means LLVM 16 supports 14, 15, and 17. Still not 16. Maybe then we should list it like suggested.

Then we need to make sure we update this just prior to a release. How about adding a pre-release checklist here https://libcxx.llvm.org/Contributing.html?

Is this still relevant? If not, let's close.

philnik updated this revision to Diff 484979.Dec 22 2022, 3:40 PM
philnik marked 3 inline comments as done.

Address comments

philnik edited the summary of this revision. (Show Details)Dec 26 2022, 7:38 PM
ldionne accepted this revision.Jan 19 2023, 9:17 AM
ldionne added inline comments.
libcxx/docs/Contributing.rst
65 ↗(On Diff #484979)

The summary and commit message seem outdated.

73 ↗(On Diff #484979)
74 ↗(On Diff #484979)
libcxx/utils/ci/buildkite-pipeline.yml
490

You need to update run-buildbot as well to remove that job.

This revision is now accepted and ready to land.Jan 19 2023, 9:17 AM
philnik updated this revision to Diff 490636.Jan 19 2023, 12:52 PM
philnik marked 3 inline comments as done.

Address comments

philnik edited the summary of this revision. (Show Details)Jan 19 2023, 12:56 PM
This revision was landed with ongoing or failed builds.Jan 19 2023, 9:04 PM
This revision was automatically updated to reflect the committed changes.