Skip to content

Reapply '[BuildSystem] Stop building for i386-watch-simulator (#77692)' #79018

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jan 31, 2025

Conversation

cyndyishida
Copy link
Contributor

* [BuildSystem] Stop building for i386-watch-simulator
In Xcode16 it is not supported.

This initially broke client projects who were still building the legacy
architecture but now that is resolved.

cyndyishida and others added 2 commits January 29, 2025 10:29
…ang#77692)'

    * [BuildSystem] Stop building for i386-watch-simulator

    In Xcode16 it is not supported.

This initially broke client projects who were still building the legacy
architecture but now thats resolved.
This was broken by swiftlang#77692.

rdar://140313372
@cyndyishida cyndyishida requested a review from edymtt January 29, 2025 18:42
@cyndyishida
Copy link
Contributor Author

@swift-ci please test

@cyndyishida
Copy link
Contributor Author

@swift-ci please test Linux

@cyndyishida cyndyishida merged commit 3232df4 into swiftlang:main Jan 31, 2025
5 checks passed
@cyndyishida cyndyishida deleted the eng/Pr-relandi386Drop branch January 31, 2025 23:04
cyndyishida added a commit to cyndyishida/llvm-project that referenced this pull request Mar 3, 2025
…m#117013)"

This reverts commit b839386 to reapply
the removal of i386.

It previously failed because the swift config still built for i386, that is
no longer true with swiftlang/swift#79018
cyndyishida added a commit to swiftlang/llvm-project that referenced this pull request Mar 4, 2025
…m#117013)"

This reverts commit b839386 to reapply
the removal of i386.

It previously failed because the swift config still built for i386, that is
no longer true with swiftlang/swift#79018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants