-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Use mergeable symbols in embedded #8654
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
To workaround proper support for the Embedded Swift linkage model in SwiftPM, this PR updates the build plan to include -Xfrontend -mergeable symbols when building Embedded targets. This is a workaround/stop-gap solution until we can implement #8653. Fixes: 8648
|
@swift-ci please test |
dschaefer2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks!
|
@rauhul Recently, I've been seeing the following warning every time I build an Embedded Swift project. Is the Related PR: swiftlang/swift#84161 |
|
IIUC it's no longer necessary, but I'm not sure if this change can be cleanly reverted, or is a different kind of cleanup needed |
|
We should be able to easily remove this flag now. |
|
Thank you so much! 🙏 |
To workaround proper support for the Embedded Swift linkage model in SwiftPM, this PR updates the build plan to include -Xfrontend -mergeable symbols when building Embedded targets. This is a workaround/stop-gap solution until we can implement #8653.
Fixes: #8648