-
Notifications
You must be signed in to change notification settings - Fork 341
[lldb] Resolve typealias information for assiciated types via reflection #9768
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
[lldb] Resolve typealias information for assiciated types via reflection #9768
Conversation
050d894
to
a269a9b
Compare
@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.
Looks good to me. We should measure the performance and maybe come up with a better API on TypeRefBuilder to query conformances, since parsing all of them could be very expensive.
if (!reflection_ctx) | ||
return {}; | ||
|
||
Progress progress("Downloading Swift conformances"); |
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.
Maybe "parsing" instead of "downloading"?
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.
+1, or "Loading"
lldb/source/Plugins/LanguageRuntime/Swift/SwiftLanguageRuntimeDynamicTypeResolution.cpp
Outdated
Show resolved
Hide resolved
lldb/source/Plugins/LanguageRuntime/Swift/SwiftLanguageRuntimeDynamicTypeResolution.cpp
Outdated
Show resolved
Hide resolved
lldb/source/Plugins/LanguageRuntime/Swift/SwiftLanguageRuntimeDynamicTypeResolution.cpp
Outdated
Show resolved
Hide resolved
if (!reflection_ctx) | ||
return {}; | ||
|
||
Progress progress("Downloading Swift conformances"); |
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.
+1, or "Loading"
7c3ac2f
to
3199d3e
Compare
In this testcase there is an associated type for which no debug info is generated, since there is neither function nor a variable holding on to it. However, the associated type information is stored in the Witness Table, which we can query in the runtime. rdar://140940434
3199d3e
to
a098c18
Compare
@swift-ci test |
In this testcase there is an associated type for which no debug info
is generated, since there is neither function nor a variable holding
on to it. However, the associated type information is stored in the
Witness Table, which we can query in the runtime.
rdar://140940434