Skip to content

AST/SILGen: Consider bridgeable value types to be bridgeable when bound to generic parameters. #2708

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

Conversation

jckarter
Copy link
Contributor

We were failing to bridge Array parameters in the signatures of Objective-C generics when their NSArray * type in ObjC depended on generic parameters. This partially fixes rdar://problem/26371959, though IRGen still needs a fix to get us all the way through (which @rjmccall is working on).

(@DougGregor did all the work here, I'm just testing and committing.)

…nd to generic parameters.

We were failing to bridge Array<T> parameters in the signatures of Objective-C generics when their NSArray<T> * type in ObjC depended on generic parameters. This partially fixes rdar://problem/26371959, though IRGen still needs a fix to get us all the way through (which @rjmccall is working on).

(@DougGregor did all the work here, I'm just testing and committing.)
@jckarter
Copy link
Contributor Author

@swift-ci Please test and merge

@jckarter
Copy link
Contributor Author

Linux failure looks unrelated.

@jckarter jckarter merged commit 87d73e9 into swiftlang:master May 26, 2016
@jckarter jckarter deleted the bridging-objc-generic-containers branch May 26, 2016 02:32
@jtbandes
Copy link
Contributor

Interesting. Is this perhaps related to https://bugs.swift.org/browse/SR-772 ?

@jckarter
Copy link
Contributor Author

I don't think that's directly related. In that case we know concretely we need an Array<() -> ()>.

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