-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[move-function] Add support for properly checking let x: Int without an initial value. #40499
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
[move-function] Add support for properly checking let x: Int without an initial value. #40499
Conversation
@swift-ci smoke test |
@swift-ci smoke test linux platform |
1 similar comment
@swift-ci smoke test linux platform |
I looked into the Linux failure! It is actually b/c we run DestroyHoisting. I am just going to manually disable DestroyHoisting on this test since eventually we are going to run it not in the mandatory pipeline and that is what I am testing here. |
…an initial value. We process these as loadable vars. This is really useful since it ensures that uniqueness is preserved in this case: ``` let x: K2 do { x = self.k2 } switch _move(x)[userHandle] { case .foo: assert(_isUnique(&self.k2)) } ``` I added a test that proves this.
393937f
to
396f510
Compare
@swift-ci test |
Build failed |
Build failed |
@swift-ci test |
Build failed |
The smoke test signal bot timed out @najacque . I think if I just smoke test it should be fine. |
@swift-ci smoke test linux platform |
The actual linux test succeeded though, so I fixed the failed test. |
Build failed |
MacOS timed out as well |
@swift-ci smoke test macOS platform |
We process these as loadable vars. This is really useful since it ensures that
uniqueness is preserved in this case:
I added a test that proves this.