Skip to content

[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

Merged
merged 4 commits into from
Mar 20, 2025

Conversation

Xazax-hun
Copy link
Contributor

We do not have dynamic self metadata for foreign reference types.

rdar://145066864

@Xazax-hun Xazax-hun added the c++ interop Feature: Interoperability with C++ label Feb 27, 2025
@Xazax-hun Xazax-hun force-pushed the gaborh/dynamic-self-frt branch 2 times, most recently from 905ebdf to 15731c3 Compare March 3, 2025 15:16
Copy link
Contributor

@j-hui j-hui left a 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.

Gabor Horvath and others added 3 commits March 19, 2025 12:56
We do not have dynamic self metadata for foreign reference types.

rdar://145066864
@Xazax-hun Xazax-hun force-pushed the gaborh/dynamic-self-frt branch from 3a287c1 to ebfde99 Compare March 19, 2025 12:57
@Xazax-hun Xazax-hun force-pushed the gaborh/dynamic-self-frt branch from ebfde99 to 8f58ad6 Compare March 20, 2025 12:36
@Xazax-hun Xazax-hun requested a review from susmonteiro as a code owner March 20, 2025 12:36
@Xazax-hun
Copy link
Contributor Author

@swift-ci please smoke test

@Xazax-hun
Copy link
Contributor Author

I'm also curious to know what's going on with the Windows CI, and why that only triggers on Windows.

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.

@Xazax-hun Xazax-hun merged commit ddd36a7 into main Mar 20, 2025
3 checks passed
@Xazax-hun Xazax-hun deleted the gaborh/dynamic-self-frt branch March 20, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ interop Feature: Interoperability with C++
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants