Skip to content

[lldb] Look through existential types when handling typealiases in SwiftASTContext to correctly handle AnyObject existentials.#4852

Closed
hborla wants to merge 1 commit intoswiftlang:stable/20211026from
hborla:lldb-existential-any-anyobject
Closed

[lldb] Look through existential types when handling typealiases in SwiftASTContext to correctly handle AnyObject existentials.#4852
hborla wants to merge 1 commit intoswiftlang:stable/20211026from
hborla:lldb-existential-any-anyobject

Conversation

@hborla
Copy link
Copy Markdown
Member

@hborla hborla commented Jun 18, 2022

swiftlang/swift#41909 updates the representation of Any and AnyObject in the Swift frontend to use ExistentialType. Typealiases like AnyObject can be used as existential types, which are represented as ExistentialType(TypeAliasType), so LLDB should look through to an existential type's constraint when checking whether it has a typealias.

SwiftASTContext to correctly handle AnyObject existentials.
@hborla
Copy link
Copy Markdown
Member Author

hborla commented Jun 18, 2022

swiftlang/swift#41909

@swift-ci please test

Copy link
Copy Markdown

@adrian-prantl adrian-prantl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@adrian-prantl adrian-prantl self-requested a review June 18, 2022 17:13
Copy link
Copy Markdown

@adrian-prantl adrian-prantl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry — just realized that this PR is on the wrong branch.
The Swift release/5.7 branch goes with llvm-project swift/release/5.7. Any commit there also needs to be (manually) cherrypicked to stable/20220421 and to next. You can push these cherry-picks without testing.

@hborla
Copy link
Copy Markdown
Member Author

hborla commented Jun 23, 2022

@adrian-prantl whoops, I'll get that fixed now, thank you!!

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.

3 participants