Skip to content

Conversation

@ldionne
Copy link
Member

@ldionne ldionne commented Nov 14, 2024

After a recent Github Actions runner policy change 1, the version of Xcode included in
the macos-14 image went from Xcode 16 to Xcode 15, breaking our build bots.

This moves the bots to the macos 15 (public preview) image, which contains Xcode 16.

Also, adjust an UNSUPPORTED annotation that was incorrectly targeting macos 13.7
when it should have been targeting a version of AppleClang.

@ldionne ldionne force-pushed the review/fix-macos-ci branch from 366b049 to 5b58ef0 Compare November 14, 2024 16:05
@ldionne ldionne marked this pull request as ready for review November 14, 2024 22:57
@ldionne ldionne requested a review from a team as a code owner November 14, 2024 22:57
@ldionne ldionne force-pushed the review/fix-macos-ci branch from cd2e52b to 6eaaa72 Compare November 14, 2024 22:59
@ldionne
Copy link
Member Author

ldionne commented Nov 14, 2024

Merging since this passed CI based on previous runs when this was a draft, and this is currently failing all macOS jobs.

@ldionne ldionne merged commit 2d6459c into llvm:main Nov 14, 2024
9 of 12 checks passed
@ldionne ldionne deleted the review/fix-macos-ci branch November 14, 2024 23:00
@ldionne
Copy link
Member Author

ldionne commented Nov 14, 2024

I guess this was merged with a proper commit message but an incorrect title, my bad.

@llvmbot llvmbot added libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. github:workflow labels Nov 16, 2024
@llvmbot
Copy link
Member

llvmbot commented Nov 16, 2024

@llvm/pr-subscribers-github-workflow

@llvm/pr-subscribers-libcxx

Author: Louis Dionne (ldionne)

Changes

After a recent Github Actions runner policy change 1, the version of Xcode included in
the macos-14 image went from Xcode 16 to Xcode 15, breaking our build bots.

This moves the bots to the macos 15 (public preview) image, which contains Xcode 16.

Also, adjust an UNSUPPORTED annotation that was incorrectly targeting macos 13.7
when it should have been targeting a version of AppleClang.


Full diff: https://github.com/llvm/llvm-project/pull/116216.diff

2 Files Affected:

  • (modified) .github/workflows/libcxx-build-and-test.yaml (+4-4)
  • (modified) libcxx/test/std/experimental/simd/simd.class/simd_unary.pass.cpp (+2-3)
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index 2184ddd49537b5..91b929732f2808 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -202,13 +202,13 @@ jobs:
       matrix:
         include:
         - config: generic-cxx03
-          os: macos-latest
+          os: macos-15
         - config: generic-cxx23
-          os: macos-latest
+          os: macos-15
         - config: generic-modules
-          os: macos-latest
+          os: macos-15
         - config: apple-configuration
-          os: macos-latest
+          os: macos-15
         - config: apple-system
           os: macos-13
         - config: apple-system-hardened
diff --git a/libcxx/test/std/experimental/simd/simd.class/simd_unary.pass.cpp b/libcxx/test/std/experimental/simd/simd.class/simd_unary.pass.cpp
index 8cc9924a3cdae1..17ecfd3b50d255 100644
--- a/libcxx/test/std/experimental/simd/simd.class/simd_unary.pass.cpp
+++ b/libcxx/test/std/experimental/simd/simd.class/simd_unary.pass.cpp
@@ -12,9 +12,8 @@
 //   Pass-by-value arguments with alignment greater than register width are not supported.
 // XFAIL: target=powerpc{{.*}}-ibm-{{.*}} && (clang-17 || clang-18)
 
-// FIXME: The following issue occurs on macOS 13.7 with Apple clang 15.0.0:
-//   clang: error: unable to execute command: Illegal instruction: 4
-// XFAIL: target=x86_64-apple-macosx13.7
+// This test crashes AppleClang 15 but not later versions.
+// UNSUPPORTED: apple-clang-15
 
 // FIXME: The following issue occurs on Windows to Armv7 Ubuntu Linux:
 //   Assertion failed: N->getValueType(0) == MVT::v1i1 && "Expected v1i1 type"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github:workflow libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants