[libcxx-commits] [PATCH] D133364: [libc++] Remove old CI configurations and update the supported compiler versions

Mon Sep 12 10:19:45 PDT 2022

Mordante added inline comments.

================
Comment at: libcxx/docs/index.rst:107
 ============ =============== ========================== =====================
-Clang        13, 14                                     latest two stable releases per `LLVM's release page <https://releases.llvm.org>`_
+Clang        14, 15, 16                                 latest two stable releases per `LLVM's release page <https://releases.llvm.org>`_ and current trunk
 AppleClang   13                                         latest stable release per `Xcode's release page <https://developer.apple.com/documentation/xcode-release-notes>`_
----------------
philnik wrote:
> Mordante wrote:
> > 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.
> 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.
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?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133364/new/

https://reviews.llvm.org/D133364