Skip to content

[5.9][ConstraintSystem] Some more variadic generic fixes #66173

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 5 commits into from
May 29, 2023

Conversation

xedin
Copy link
Contributor

@xedin xedin commented May 26, 2023

  • Explanation:

    • Limit assert in getFunctionArgApplyInfo to types without type parameter packs
    • PackMacher: Match pack expansion elements pair-wise if pack arity matches
    • Diagnose conformance failures related to pack expansion patterns
    • Add a possibility to check whether PackConformance is invalid
  • Scope: Expressions with value pack expansions.

  • Main Branch PR: [ConstraintSystem] Some more variadic generic fixes #66080

  • Risk: Very Low

  • Reviewed By: @hborla

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

Resolves: rdar://109539394

xedin added 5 commits May 26, 2023 09:29
… without type parameter packs

If function type of some declaration has a at least on type parameter
pack it could mean that number of parameters in "applied" type could
be different from that of "interface" type.

(cherry picked from commit 70e4cf0)
…ty matches

This covers situations like `Pack{repeat each T1, repeat each T2}`
vs. `Pack{repeat $T1, repeat $T2}` where type variables are allowed
to bind to packs.

Resolves: rdar://109539394
(cherry picked from commit 1e03cdc)
…h is okay in some cases

(cherry picked from commit 7c91107)
@xedin xedin added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels May 26, 2023
@xedin xedin requested a review from hborla May 26, 2023 16:31
@xedin xedin requested a review from a team as a code owner May 26, 2023 16:31
@xedin
Copy link
Contributor Author

xedin commented May 26, 2023

@swift-ci please test

@xedin xedin merged commit 1231953 into swiftlang:release/5.9 May 29, 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