Skip to content

GenericCloner: make sure that we can always specialize in mandatory mode #71314

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 2 commits into from
Feb 2, 2024

Conversation

eeckstein
Copy link
Contributor

This may involve changing the linkage of the specialized function.
If called from a serialized function we cannot make the specialized function shared and non-serialized.
The only other option is to keep the original function's linkage.

Fixes a crash in embedded mode.

rdar://121675461

@eeckstein
Copy link
Contributor Author

@swift-ci test

…ure-specialized functions

We need to keep the original linkage because it would be illegal to call a shared not-serialized function from a serialized function.

Also, rename the API to create the specialized function.
This may involve changing the linkage of the specialized function.
If called from a serialized function we cannot make the specialized function shared and non-serialized.
The only other option is to keep the original function's linkage.

rdar://121675461
@eeckstein eeckstein force-pushed the fix-mandatory-specializtion branch from 0bdd60a to 874360e Compare February 2, 2024 06:29
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test

@eeckstein eeckstein enabled auto-merge February 2, 2024 06:29
@eeckstein eeckstein merged commit 034a8a0 into swiftlang:main Feb 2, 2024
@eeckstein eeckstein deleted the fix-mandatory-specializtion branch February 2, 2024 09:10
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.

1 participant