-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[stdlib] Condition Mutex.swift on platform #74143
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
@swift-ci please smoke test |
preset=stdlib_S_standalone_minimal_macho_x86_64,build,test |
preset=stdlib_S_standalone_minimal_macho_x86_64,build,test |
Are we going to end up having to be able to check for availability? Should we put an unavailable placeholder in to give people a better error on other platforms? Also, do we need this on 6.0? |
It seems like we should be able to have an unavailable availability for Mutex on platforms that don't support it (we can compile a special version of |
@swift-ci please smoke test |
preset=stdlib_S_standalone_minimal_macho_x86_64,build,test |
[stdlib] Condition Mutex.swift on platform
Mutex.swift was previously never conditional and always built, but the handle was conditioned based on platform. In a minimal toolchain, we don't have a platform to piggyback on, so we shouldn't build Mutex.swift either.
Resolves: rdar://129287049