Skip to content

[2025 Branch][clang][Darwin] Align all OS versions for 26 #10822

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

Conversation

cyndyishida
Copy link

No description provided.

…lvm#142013)

To be able to handle all of the ways the platform & deployment version
can be represented in command line flags, the Darwin toolchain holds a
type `DarwinPlatform` to help represent them. This patch simplifies the
logic by:
* reducing the amount of work done between string & version tuples
conversions
* renaming variables to reduce confusion about what target triple
information is being manipulated.
* allowing implicit transformation of macOS10.16 -> 11, there are other
places in the compiler where this happens, and it was a bit confusing
that the driver didn't do that for the cc1 call.

This is not a major refactor, but more simple & common tweaks across the
file, in hopes to make it more readable.
* Translate the following versions to 26.
  * watchOS 12 -> 26
  * visionOS 3 -> 26
  * macos 16 -> 26
  * iOS 19 -> 26
  * tvOS 19 -> 26

* Emit diagnostics, but allow conversion when clients attempt to use invalid gaps in
OS versioning in availability.

* For target-triples, only allow "valid" versions for implicit
conversions.
@cyndyishida
Copy link
Author

@swift-ci please test LLVM

@cyndyishida
Copy link
Author

@swift-ci please smoke test

@cyndyishida
Copy link
Author

@swift-ci please smoke test windows

@cyndyishida
Copy link
Author

@swift-ci please test

@cyndyishida cyndyishida merged commit bdb6ad6 into swiftlang:stable/20250601 Jun 12, 2025
2 of 5 checks passed
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.

1 participant