Skip to content

Conversation

@gottesmm
Copy link
Contributor

@gottesmm gottesmm commented Aug 6, 2019

Now we just always mark functions as having or not having ownership. So we know
when parsing/deserializing what state we are in already. There are also
assertions in SILBuilder around most of the instructions that can exist in only
a single mode, meaning even the error was sort of redundant.

Now we just always mark functions as having or not having ownership. So we know
when parsing/deserializing what state we are in already. There are also
assertions in SILBuilder around most of the instructions that can exist in only
a single mode, meaning even the error was sort of redundant.
@gottesmm
Copy link
Contributor Author

gottesmm commented Aug 6, 2019

I think this may need another out of tree patch I have. But I think it might be independent (lets see...).

@gottesmm
Copy link
Contributor Author

gottesmm commented Aug 6, 2019

@swift-ci smoke test

@gottesmm
Copy link
Contributor Author

gottesmm commented Aug 6, 2019

@swift-ci smoke test linux platform

@gottesmm gottesmm merged commit 037f6c1 into swiftlang:master Aug 6, 2019
@gottesmm gottesmm deleted the pr-48526a73d520afbaaeda3525ce0a6f87a84fb1dc branch August 6, 2019 06: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.

1 participant