Skip to content

[ClangImporter] Make sure that inherited convenience constructors are included in members of IterableDeclContext #32214

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

akyrtzi
Copy link
Contributor

@akyrtzi akyrtzi commented Jun 5, 2020

Previously inherited constructors would be skipped from added in member list, depending on the order of request evaluator calls.
This was a regression compared to swift 5.2

@akyrtzi akyrtzi requested review from nkcsgexi and CodaFi June 5, 2020 21:47
@CodaFi
Copy link
Contributor

CodaFi commented Jun 5, 2020

Don't we have to ensure that this is transitive? We're not inserting this constructor into any of the many tables associated with the subclass.

@nkcsgexi nkcsgexi requested a review from beccadax June 5, 2020 22:40
@akyrtzi
Copy link
Contributor Author

akyrtzi commented Jun 6, 2020

Don't we have to ensure that this is transitive? We're not inserting this constructor into any of the many tables associated with the subclass.

It is inserted in the lookup table and Impl.ConstructorsForNominal for the subclass, or am I misunderstanding what you mean?

… included in members of `IterableDeclContext`

Previously inherited constructors would be skipped from added in member list, depending on the order of request evaluator calls.
This was a regression compared to swift 5.2
@akyrtzi akyrtzi force-pushed the clang-include-inherited-convenience-inits-for-members branch from a50b982 to 915e215 Compare June 10, 2020 20:23
@swiftlang swiftlang deleted a comment from swift-ci Jun 10, 2020
@swiftlang swiftlang deleted a comment from swift-ci Jun 10, 2020
@akyrtzi
Copy link
Contributor Author

akyrtzi commented Jun 10, 2020

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - a50b9828eae7540763dfa41062483e517e5660cc

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - a50b9828eae7540763dfa41062483e517e5660cc

Copy link
Contributor

@beccadax beccadax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@akyrtzi akyrtzi merged commit a3ce462 into swiftlang:master Jun 10, 2020
@akyrtzi akyrtzi deleted the clang-include-inherited-convenience-inits-for-members branch June 10, 2020 23:29
akyrtzi added a commit that referenced this pull request Jun 16, 2020
akyrtzi added a commit that referenced this pull request Jun 16, 2020
…nvenience-inits-for-members

[ClangImporter] Follow-up for #32214, fix assertion hit due to missing check
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.

4 participants