Skip to content

[silgen] When forwarding an rvalue into memory to pass as an argument… #15131

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

Conversation

gottesmm
Copy link
Contributor

…, make sure it is at plus 1.

This can only come up with the +1 runtime if we need to pass a +0 value into
memory. As with the other ArgumentSource change, I don't think such code can be
emitted today. It does happen often with the +0 runtime though.

rdar://34222540

…, make sure it is at plus 1.

This can only come up with the +1 runtime if we need to pass a +0 value into
memory. As with the other ArgumentSource change, I don't think such code can be
emitted today. It does happen often with the +0 runtime though.

rdar://34222540
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test linux platform

1 similar comment
@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test linux platform

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test linux platform

3 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test linux platform

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test linux platform

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test linux platform

@gottesmm gottesmm merged commit 5eff528 into swiftlang:master Mar 10, 2018
@gottesmm gottesmm deleted the pr-f55b282e4fa6433d6c2d5d810e239930e4fa87b5 branch March 10, 2018 19:03
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