Skip to content

[SE-0408] Enable nested iteration #70196

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 2 commits into from
Jan 27, 2024

Conversation

simanerush
Copy link
Member

@simanerush simanerush commented Dec 4, 2023

Enables nested iteration for SE-0408

Changes:

  • Cache the generic environment associated with the outer loop in the constraint system and apply it in getPackElementEnvironment
  • In mapElementTypeIntoPackContext, use the interface type to obtain the correct depth.

@ahoppen
Copy link
Member

ahoppen commented Dec 4, 2023

Thanks for also adding repeat to the code completion results for loops 🙏🏽

@simanerush simanerush force-pushed the nested-pack-iteration branch from eff5640 to cfe918f Compare December 8, 2023 06:00
@simanerush simanerush force-pushed the nested-pack-iteration branch from cfe918f to 5a74d5c Compare January 6, 2024 19:51
@simanerush simanerush requested a review from xedin January 6, 2024 19:51
@simanerush
Copy link
Member Author

@swift-ci please smoke test

Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

Looks good, I left a few minor and refactoring comments inline.

@simanerush simanerush force-pushed the nested-pack-iteration branch from 5a74d5c to 9b6fa6c Compare January 27, 2024 02:54
@simanerush simanerush requested a review from xedin January 27, 2024 02:55
@xedin
Copy link
Contributor

xedin commented Jan 27, 2024

Feel free to land this!

… with`for`-`in` loops over parameter packs to apply in `getPackElementEnvironment`.
@simanerush simanerush force-pushed the nested-pack-iteration branch from 9b6fa6c to 0b167b5 Compare January 27, 2024 08:12
@simanerush
Copy link
Member Author

@swift-ci Please smoke test

@simanerush simanerush merged commit 066f253 into swiftlang:main Jan 27, 2024
@simanerush simanerush deleted the nested-pack-iteration branch March 3, 2024 05:43
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