-
Notifications
You must be signed in to change notification settings - Fork 10.5k
SILGen: Don't emit key path descriptors for properties with mutating getters [4.2] #15250
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
SILGen: Don't emit key path descriptors for properties with mutating getters [4.2] #15250
Conversation
…getters Fixes <https://bugs.swift.org/browse/SR-7176>, <rdar://problem/38439418>.
@swift-ci Please test |
Build failed |
@parkera What happened here?
|
I don’t think we’ve changed this recently. Was there a change on String? |
@milseman Any ideas? |
@parkera In swift-4.2-branch, String.hasSuffix() is not defined on Linux. On master, it's implemented on both platforms. Can you remove the usage from Foundation? |
The breakage was introduced by PR swiftlang/swift-corelibs-foundation#1404 |
@parkera Actually it looks like the method was removed from the Swift stdlib so this is our fault. rdar://38424188 |
@airspeedswift changed the hasPrefix/hasSuffix code recently. Is it available on Linux? |
Thanks Slava! |
It's that the PR that added them to the stdlib for Linux needed picking to 4.2 because corelibs-foundation doesn't have a 4.2 branch. Testing #15258 now. |
@swift-ci Please test Linux |
Fixes https://bugs.swift.org/browse/SR-7176, rdar://problem/38439418.