Skip to content

Add attr to enable lexical lifetime per function. #63766

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

nate-chandler
Copy link
Contributor

Enables modules to migrate to the new lifetime rules incrementally.

@nate-chandler nate-chandler force-pushed the lexical_lifetimes/per-function branch 4 times, most recently from 584c7ab to fc1119b Compare February 18, 2023 17:47
Enables modules to migrate to the new lifetime rules incrementally.
@nate-chandler nate-chandler force-pushed the lexical_lifetimes/per-function branch from fc1119b to ec1a5e0 Compare February 18, 2023 20:00
@nate-chandler
Copy link
Contributor Author

@swift-ci please 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.

Sema changes LGTM :)

@nate-chandler nate-chandler merged commit 77feef5 into swiftlang:main Feb 22, 2023
@nate-chandler nate-chandler deleted the lexical_lifetimes/per-function branch February 22, 2023 20:33
Copy link
Contributor

@atrick atrick left a comment

Choose a reason for hiding this comment

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

All changes look good. Thanks for doing this. It'll be nice to play with even if we don't end up using it. I notice that we still appear to support _noEagerMove too, so we have all the controls.

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