Skip to content

Runtime: Fix breakage when generic subclasses directly inherit NSObject #4877

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

slavapestov
Copy link
Contributor

No description provided.

@slavapestov
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - f79454eb5a896943dbd09d43012c0444211fc599
Test requested by - @slavapestov

Remove special-casing that makes NSObject fragile since
this messes up layout. The optimization probably has little
practical benefit anyway.

Fixes <https://bugs.swift.org/browse/SR-2586>.
@slavapestov slavapestov force-pushed the fix-generic-subclass-nsobject branch from f79454e to 584316d Compare September 20, 2016 23:08
@slavapestov
Copy link
Contributor Author

@swift-ci Please test

1 similar comment
@slavapestov
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 584316d
Test requested by - @slavapestov

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 584316d
Test requested by - @slavapestov

@gparker42
Copy link
Contributor

Test failure looks like #4893.

@slavapestov
Copy link
Contributor Author

@swift-ci Please test and merge

1 similar comment
@slavapestov
Copy link
Contributor Author

@swift-ci Please test and merge

@swift-ci swift-ci merged commit 1eaa0c1 into swiftlang:master Sep 21, 2016
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.

3 participants