[libcxx-commits] [PATCH] D156922: [libc++][CI] Upgrades to LLVM 18 as HEAD version.

Sat Aug 19 08:22:14 PDT 2023

philnik added inline comments.

================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:302

   - label: "Clang 15"
     command: "libcxx/utils/ci/run-buildbot generic-cxx23"
----------------
Mordante wrote:
> philnik wrote:
> > Is there a reason we keep 15 around? Also, could we make these changes automatic like we do for GCC?
> We still officially support clang-15. The main idea is to make sure we still work with clang-15 so we can be sure backported patches work on clang-15. Once LLVM 17 is released this can be removed.
> 
> What do you mean with automatically?
This doesn't really make sense to me. This would require us to keep 15 until we are branching for LLVM 18, since there will be fixes backported until then. IMO the right thing to make this working properly is to run the CI on the release branch when a libc++ patch is backported.

I mean that we use `label: "Clang ${LLVM_STABLE_VERSION}-1"` (or however you have to write it to make it work) and then just update the version at the top of the file.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156922