-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[cxx-interop] Interpret Self as a static shorthand for FRTs #79662
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
905ebdf
to
15731c3
Compare
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.
Overall this LGTM. Left a nit and a question.
I'm also curious to know what's going on with the Windows CI, and why that only triggers on Windows.
716ae55
to
3a287c1
Compare
We do not have dynamic self metadata for foreign reference types. rdar://145066864
Co-authored-by: Anthony Latsis <[email protected]>
3a287c1
to
ebfde99
Compare
Co-authored-by: John Hui <[email protected]>
ebfde99
to
8f58ad6
Compare
@swift-ci please smoke test |
It looks like the root cause of the problem is that we do not support emitting metadata for some foreign types on Windows. This is a known limitation and the crash is not related to my patch, the code crashes the compiler even before my changes on Windows. Disabling the test on Windows for now. |
We do not have dynamic self metadata for foreign reference types.
rdar://145066864