Skip to content

Conversation

ldionne
Copy link
Member

@ldionne ldionne commented Sep 14, 2023

We were running the test suite in C++20 mode, but we should really be tracking the latest supported version. We can't do C++26 because the latest stable AppleClang doesn't support it yet.

We were running the test suite in C++20 mode, but we should really
be tracking the latest supported version. We can't do C++26 because
the latest stable AppleClang doesn't support it yet.
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Sep 14, 2023
@llvmbot
Copy link
Member

llvmbot commented Sep 14, 2023

@llvm/pr-subscribers-libcxx

Changes We were running the test suite in C++20 mode, but we should really be tracking the latest supported version. We can't do C++26 because the latest stable AppleClang doesn't support it yet. -- Full diff: https://github.com//pull/66367.diff

1 Files Affected:

  • (modified) libcxx/utils/ci/buildkite-pipeline.yml (+2-2)
diff --git a/libcxx/utils/ci/buildkite-pipeline.yml b/libcxx/utils/ci/buildkite-pipeline.yml
index 90bb343f7c7ad03..f2329457c7bd938 100644
--- a/libcxx/utils/ci/buildkite-pipeline.yml
+++ b/libcxx/utils/ci/buildkite-pipeline.yml
@@ -908,7 +908,7 @@ steps:
   - group: ":mac: Apple"
     steps:
     - label: "MacOS x86_64"
-      command: "libcxx/utils/ci/run-buildbot generic-cxx20"
+      command: "libcxx/utils/ci/run-buildbot generic-cxx23"
       artifact_paths:
         - "**/test-results.xml"
         - "**/*.abilist"
@@ -923,7 +923,7 @@ steps:
       timeout_in_minutes: 120
 
     - label: "MacOS arm64"
-      command: "libcxx/utils/ci/run-buildbot generic-cxx20"
+      command: "libcxx/utils/ci/run-buildbot generic-cxx23"
       artifact_paths:
         - "**/test-results.xml"
         - "**/*.abilist"

Copy link
Member

@mordante mordante left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM!

@ldionne
Copy link
Member Author

ldionne commented Sep 15, 2023

The CI failure was on Windows and looks clearly unrelated.

@ldionne ldionne merged commit 320d4c9 into llvm:main Sep 15, 2023
ZijunZhaoCCK pushed a commit to ZijunZhaoCCK/llvm-project that referenced this pull request Sep 19, 2023
)

We were running the test suite in C++20 mode, but we should really be
tracking the latest supported version. We can't do C++26 because the
latest stable AppleClang doesn't support it yet.
@ldionne ldionne deleted the review/update-ci-jobs-cxx20 branch January 30, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants