Skip to content

[Bridging PCH] Teach SourceKit to ignore the bridging-pch driver options #7217

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 Feb 3, 2017

SourceKit unfortunately gets upset when users pass -{enable,disable}-bridging-pch via OTHER_SWIFT_FLAGS. This patch adds these driver-only options to SourceKit's local
ignore-list.

rdar://30342163

@graydon
Copy link
Contributor Author

graydon commented Feb 3, 2017

@swift-ci please smoke test

Copy link
Contributor

@benlangmuir benlangmuir left a comment

Choose a reason for hiding this comment

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

LGTM

@graydon
Copy link
Contributor Author

graydon commented Feb 3, 2017

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Feb 3, 2017

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

@DougGregor
Copy link
Member

I've never seen this before...

/Applications/Xcode-beta.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS10.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CVOpenGLESTexture.h:23:2: error: malformed or corrupted AST file: 'SourceLocation remap refers to unknown module'

@tkremenek
Copy link
Member

@DougGregor This looks like a Clang issue.

@tkremenek
Copy link
Member

@swift-ci test macOS

@swift-ci
Copy link
Contributor

swift-ci commented Feb 3, 2017

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 81b3c0d
Test requested by - @tkremenek

@tkremenek
Copy link
Member

@swift-ci clean test macOS

@ematejska ematejska merged commit 0502fb8 into swiftlang:swift-3.1-branch Feb 3, 2017
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.

6 participants