Skip to content

[5.9][ConstraintSystem] Fix a couple of issues related to generic specialization #67026

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
merged 1 commit into from
Jun 30, 2023

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Jun 29, 2023

Cherry-pick of #66971


  • Explanation:

    • Teach specialization constraint to look through non-generic typealiases
    • Detect and diagnose attempts to specialize non-generic types/aliases
    • Detect and diagnose attempts to specialize types with invalid number of arguments
  • Scope: Expressions with generic specialization of typealiases that are shadowed by macros

  • Main Branch PR: [ConstraintSystem] Fix a couple of issues related to generic specialization #66971

  • Risk: Low

  • Reviewed By: @hborla @slavapestov

  • Testing: Added regression test-cases to the suite.

Resolves: rdar://111239949

(cherry picked from commit 42f80fb)
(cherry picked from commit cbaf558)
(cherry picked from commit ab2f47b)
(cherry picked from commit 40169c7)
(cherry picked from commit 2bbda09)
(cherry picked from commit cd057bb)
(cherry picked from commit be4df5a)
(cherry picked from commit b9b21fc)
(cherry picked from commit de729e2)

…zation

- Teach specialization constraint to look through non-generic typealiases
- Detect and diagnose attempts to specialize non-generic types/aliases
- Detect and diagnose attempts to specialize types with invalid number of arguments

(cherry picked from commit 42f80fb)
(cherry picked from commit cbaf558)
(cherry picked from commit ab2f47b)
(cherry picked from commit 40169c7)
(cherry picked from commit 2bbda09)
(cherry picked from commit cd057bb)
(cherry picked from commit be4df5a)
(cherry picked from commit b9b21fc)
(cherry picked from commit de729e2)
@xedin xedin added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels Jun 29, 2023
@xedin xedin requested a review from hborla June 29, 2023 19:30
@xedin xedin requested a review from a team as a code owner June 29, 2023 19:30
@xedin
Copy link
Contributor Author

xedin commented Jun 29, 2023

@swift-ci please test

@xedin xedin merged commit fab7628 into swiftlang:release/5.9 Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.9
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants