Skip to content

SIL: KeyPathInst has to have side effects, since it retains. #9263

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
merged 1 commit into from
May 4, 2017

Conversation

jckarter
Copy link
Contributor

@jckarter jckarter commented May 4, 2017

Fixes rdar://problem/31975279.

@jckarter
Copy link
Contributor Author

jckarter commented May 4, 2017

@swift-ci Please smoke test

@jckarter
Copy link
Contributor Author

jckarter commented May 4, 2017

preset=buildbot,tools=RA,stdlib=RD
@swift-ci Please test macOS with preset

@jckarter
Copy link
Contributor Author

jckarter commented May 4, 2017

@shahmishal
Copy link
Member

Looks good!

preset=buildbot,tools=RA,stdlib=RD
@swift-ci Please test macOS with preset

@shahmishal
Copy link
Member

preset=buildbot,tools=RA,stdlib=RD
@swift-ci Please test macOS with preset

@jckarter
Copy link
Contributor Author

jckarter commented May 4, 2017

@shahmishal Will the status show up in Github, or do I have to look at ci.swift.org?

@jckarter jckarter merged commit 7bb2064 into swiftlang:master May 4, 2017
@shahmishal
Copy link
Member

shahmishal commented May 4, 2017

@shahmishal Will the status show up in Github, or do I have to look at ci.swift.org?

It should show up on GitHub, I am going to look into it.

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