Skip to content

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

Conversation

slavapestov
Copy link
Contributor

Fixes https://bugs.swift.org/browse/SR-7176, rdar://problem/38439418.

@slavapestov
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 96dc765

@slavapestov
Copy link
Contributor Author

@parkera What happened here?

Foundation/NSArray.swift:670:20: error: value of type 'String' has no member 'hasSuffix'
19:07:33                 if filename.hasSuffix(ext) && filename.count > ext.count {
19:07:33                    ^~~~~~~~ ~~~~~~~~~

@parkera
Copy link
Contributor

parkera commented Mar 15, 2018

I don’t think we’ve changed this recently. Was there a change on String?

@slavapestov
Copy link
Contributor Author

@milseman Any ideas?

@slavapestov
Copy link
Contributor Author

@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?

@slavapestov
Copy link
Contributor Author

slavapestov commented Mar 15, 2018

The breakage was introduced by PR swiftlang/swift-corelibs-foundation#1404

@slavapestov
Copy link
Contributor Author

@parkera Actually it looks like the method was removed from the Swift stdlib so this is our fault. rdar://38424188

@milseman
Copy link
Member

@airspeedswift changed the hasPrefix/hasSuffix code recently. Is it available on Linux?

@jckarter
Copy link
Contributor

Thanks Slava!

@airspeedswift
Copy link
Member

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.

@slavapestov
Copy link
Contributor Author

@swift-ci Please test Linux

@slavapestov slavapestov merged commit 148a3a5 into swiftlang:swift-4.2-branch Mar 15, 2018
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.

6 participants