Skip to content

[Bridging PCH] Serialize diagnostics in -emit-pch step. #7944

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

graydon
Copy link
Contributor

@graydon graydon commented Mar 6, 2017

The driver currently doesn't serialize any diagnostics in the generate-pch job type. It needs to if users are to get any useful feedback in Xcode when there's an error in their bridging header.

rdar://30879533

@graydon graydon requested a review from jrose-apple March 6, 2017 23:13
@graydon
Copy link
Contributor Author

graydon commented Mar 6, 2017

@swift-ci please test

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@swift-ci
Copy link
Contributor

swift-ci commented Mar 7, 2017

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fae468f
Test requested by - @graydon

@graydon graydon merged commit 26873f8 into swiftlang:master Mar 7, 2017
@graydon
Copy link
Contributor Author

graydon commented Mar 7, 2017

(There was a transient test-failure but I re-ran and it passed, so merged)

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.

3 participants