-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Extend internal _mangledTypeName()
function to take non-copyable types.
#76041
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
Extend internal _mangledTypeName()
function to take non-copyable types.
#76041
Conversation
@swift-ci test |
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.
I presume this will change again soon for ~Escapable
?
Hmmm.... Test "demangleToMetadata" crashed on Linux. |
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. Have you verified that _mangledTypeName
does in fact have the same symbol name?
And while you're in here, could you also make it accept ~Escapable
? I assume we'll want that at some point and might as well do it now.
@mikeash we have a lovely ABI test that tracks changes to stdlib symbols now, so that will fail if the symbol does change |
Not yet! |
I believe this is a known compiler bug that's filed under another radar right now. This change may be blocked on that fix. |
@swift-ci test |
…pes. `_mangledTypeName()` returns the Swift-mangled typename of a given type. This PR extends it to take non-copyable types as well. Resolves rdar://134278607.
0f1c02c
to
6390152
Compare
@swift-ci test |
_mangledTypeName()
returns the Swift-mangled typename of a given type. This PR extends it to take non-copyable types as well.Related to our use case in Swift Testing, but not to the PR directly: #69146 #69147 #71112
Resolves rdar://134278607.