Skip to content

Fix the demangling of sending result types when combined with isolation #78576

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

rjmccall
Copy link
Contributor

Fixes the immediate problem, but the presence of demangling code in the runtime means that we'll need a follow-up to fix the compiler so that it doesn't try to use the demangler to materialize metadata for function types that have both isolation and a sending result.

rdar://142443925

Fixes the immediate problem, but the presence of demangling code in the
runtime means that we'll need a follow-up to fix the compiler so that it
doesn't try to use the demangler to materialize metadata for function types
that have both isolation and a sending result.

rdar://142443925
@rjmccall rjmccall requested review from mikeash, al45tair and a team as code owners January 11, 2025 02:15
@rjmccall
Copy link
Contributor Author

@swift-ci Please test

@rjmccall
Copy link
Contributor Author

@swift-ci Please test Linux

@rjmccall rjmccall merged commit 11ff5b4 into swiftlang:main Jan 11, 2025
5 checks passed
@AnthonyLatsis
Copy link
Collaborator

Does this resolve #78582 or #75523?

@gottesmm
Copy link
Contributor

gottesmm commented Jan 11, 2025

@AnthonyLatsis if it isn't this, there is an additional bug that I am fixing that has to do with function types/sending results that I don't think is fixed by this. I already have a patch... I would just wait for Monday before you dig in if this doesn't fix those issues... it is the weekend after all ; p

@AnthonyLatsis
Copy link
Collaborator

No worries, I was just triaging recent issues and spotted this PR right next to one of them in my inbox. Thanks for letting me know, though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants