Skip to content

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