Skip to content

DispatchStubs: make more Windows friendly #415

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 1 commit into from
Nov 5, 2018

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Nov 2, 2018

Provide an alias for the import symbol as the ObjC runtime is normally
in a separate DLL and the compiler will annotate the function as being
DLLImport. This allows us to resolve the symbols when building the SDK
overlay.

Provide an alias for the import symbol as the ObjC runtime is normally
in a separate DLL and the compiler will annotate the function as being
DLLImport.  This allows us to resolve the symbols when building the SDK
overlay.
@compnerd
Copy link
Member Author

compnerd commented Nov 2, 2018

CC: @gparker42

@compnerd
Copy link
Member Author

compnerd commented Nov 2, 2018

@swift-ci please test

@gparker42
Copy link

Should objc_retainAutoreleasedReturnValue be defined as __declspec(dllexport)? I suspect that calls to it are emitted inside code other than libdispatch itself.

@compnerd
Copy link
Member Author

compnerd commented Nov 2, 2018

Yeah, but I expect that to be handled by DISPATCH_RUNTIME_STDLIB_INTERFACE no? Those are interfaces that should be exposed right?

@ktopley-apple ktopley-apple merged commit 9852dcb into swiftlang:master Nov 5, 2018
@compnerd compnerd deleted the stubs branch November 5, 2018 16:48
rokhinip pushed a commit that referenced this pull request Nov 5, 2021
DispatchStubs: make more Windows friendly
Signed-off-by: Kim Topley <ktopley@apple.com>
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