Skip to content
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

Demangler: Add option to omit closure signatures (#73331) #73353

Conversation

kastiglione
Copy link
Contributor

@kastiglione kastiglione commented Apr 30, 2024

Add a new demangler option which excludes a closure's type signature.

This will be used in lldb.

Closures are not subject to overloading, and so the signature will never be needed to
disambiguate. A demangled closure is uniquely identifiable by its index(es) and parent.

Where opaque types are involved, the concrete type signature can be quite complex. This
demangling option allows callers to avoid printing the underlying complex nested
concrete types.

Example:

before: closure #1 (Swift.Int) -> () in closure #1 (Swift.Int) -> () in main
after: closure #1 in closure #1 in main

Cherry picked from #73331

Add a new demangler option which excludes a closure's type signature.

This will be used in lldb.

Closures are not subject to overloading, and so the signature will never be used to 
disambiguate. A demangled closure is uniquely identifiable by its index(s) and parent.

Where opaque types are involved, the concrete type signature can be quite complex. This 
demangling option allows callers to avoid printing the underlying complex nested 
concrete types.

Example:

before: `closure #1 (Swift.Int) -> () in closure #1 (Swift.Int) -> () in main`
after: `closure #1 in closure #1 in main`
@kastiglione kastiglione requested a review from a team as a code owner April 30, 2024 20:44
@kastiglione
Copy link
Contributor Author

@swift-ci test

@kastiglione
Copy link
Contributor Author

@swift-ci smoke test

@kastiglione
Copy link
Contributor Author

@swift-ci test

@kastiglione kastiglione merged commit a85dbaf into release/6.0 May 1, 2024
5 checks passed
@kastiglione kastiglione deleted the dl/Demangler-Add-option-to-omit-closure-signatures-73331 branch May 1, 2024 17:03
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.

None yet

3 participants