Skip to content

[PackageCMO] Workaround for static witness thunk linker issue. #74531

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
Jun 19, 2024

Conversation

elsh
Copy link
Contributor

@elsh elsh commented Jun 18, 2024

PR #74468 rolled back conformance serialized kind to IsNotSerialized for non-public decls. This caused static witness thunks of package protocol to be removed while use site expects them, causing linker issues. This PR provides a workaround until a proper fix is made.

rdar://130103572

@elsh
Copy link
Contributor Author

elsh commented Jun 18, 2024

@swift-ci smoke test

PR #74468 rolled back conformance serialized kind to IsNotSerialized
for non-public decls. This caused static witness thunks to be
removed while use site expects them, causing linker issues.
This PR provides a workaround until a proper fix is made.

rdar://130103572
@elsh elsh force-pushed the elsh/static-witness-linker-workaround branch from 09e40fd to c94ba25 Compare June 18, 2024 23:24
@elsh
Copy link
Contributor Author

elsh commented Jun 18, 2024

@swift-ci smoke test

@elsh elsh merged commit 2465200 into main Jun 19, 2024
3 checks passed
@elsh elsh deleted the elsh/static-witness-linker-workaround branch June 19, 2024 04:23
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