Skip to content

IRGen: Fix assert #15074

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

Conversation

aschwaighofer
Copy link
Contributor

We only need a contextPtr if the function type requires a context

rdar://38241155
SR-7138

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Mar 8, 2018

Build failed
Swift Test Linux Platform
Git Sha - f3c7291d4266861fef0044b9325da0fa0474eb7b

@swift-ci
Copy link
Contributor

swift-ci commented Mar 8, 2018

Build failed
Swift Test OS X Platform
Git Sha - f3c7291d4266861fef0044b9325da0fa0474eb7b

We only need a contextPtr if the function type requires a context

rdar://38241155
SR-7138
@aschwaighofer aschwaighofer force-pushed the irgen_fix_closure_assert branch from f3c7291 to f0bf9af Compare March 8, 2018 13:59
@aschwaighofer
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Mar 8, 2018

Build failed
Swift Test OS X Platform
Git Sha - f3c7291d4266861fef0044b9325da0fa0474eb7b

@swift-ci
Copy link
Contributor

swift-ci commented Mar 8, 2018

Build failed
Swift Test Linux Platform
Git Sha - f3c7291d4266861fef0044b9325da0fa0474eb7b

@aschwaighofer aschwaighofer merged commit f873444 into swiftlang:master Mar 8, 2018
@aschwaighofer
Copy link
Contributor Author

Test case here: #15076

@aschwaighofer
Copy link
Contributor Author

@atrick You should be able to re-apply your SILCombiner change now. At least the IRGen failure should be fixed.

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