Skip to content

Conversation

@gottesmm
Copy link
Contributor

This is used to teach the checker that the thing being checked is supposed to be uninitialized at the mark_must_check point so that we don't put a destroy_addr there.

The way this is implemented is that we always initially add assignable_but_not_consumable but in DI once we discover that the assign we are guarding is an init, we convert the assignable to its initable variant.

rdar://106525988

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm gottesmm force-pushed the pr-08a1f1872f4e90eb29be0ba7c845d07fbb008571 branch from 43afd53 to b8efb52 Compare March 28, 2023 23:37
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@jckarter jckarter force-pushed the pr-08a1f1872f4e90eb29be0ba7c845d07fbb008571 branch from b8efb52 to 7270131 Compare April 1, 2023 00:28
…able.

This is used to teach the checker that the thing being checked is supposed to be
uninitialized at the mark_must_check point so that we don't put a destroy_addr
there.

The way this is implemented is that we always initially add
assignable_but_not_consumable but in DI once we discover that the assign we are
guarding is an init, we convert the assignable to its initable variant.

rdar://106525988
@jckarter jckarter force-pushed the pr-08a1f1872f4e90eb29be0ba7c845d07fbb008571 branch from 7270131 to 43d8ab2 Compare April 1, 2023 00:33
@jckarter
Copy link
Contributor

jckarter commented Apr 1, 2023

@swift-ci Please test

@jckarter
Copy link
Contributor

jckarter commented Apr 1, 2023

@swift-ci Please smoke test

@jckarter jckarter merged commit e5ea789 into swiftlang:main Apr 2, 2023
@gottesmm gottesmm deleted the pr-08a1f1872f4e90eb29be0ba7c845d07fbb008571 branch May 29, 2023 02:41
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.

2 participants