Skip to content

[Build] Turn off -undefined-version when building with lld #8863

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
Jun 7, 2024

Conversation

al45tair
Copy link

@al45tair al45tair commented Jun 6, 2024

Newer lld defaults to -no-undefined-version, which breaks the build because of various symbols that are (a) versioned and (b) not always defined (depending on platform).

rdar://123436280

Newer lld defaults to `-no-undefined-version`, which breaks the build
because of various symbols that are (a) versioned and (b) not always
defined (depending on platform).

rdar://123436280
@al45tair
Copy link
Author

al45tair commented Jun 6, 2024

This is a cherry pick of #8255.

@al45tair al45tair merged commit 03ca228 into swiftlang:next Jun 7, 2024
@nico
Copy link

nico commented Mar 7, 2025

(This breaks building with -DLLVM_ENABLE_LLD=ON on macOS. Maybe this should grow an AND NOT APPLE AND NOT WIN32?)

@al45tair
Copy link
Author

(This breaks building with -DLLVM_ENABLE_LLD=ON on macOS. Maybe this should grow an AND NOT APPLE AND NOT WIN32?)

I don't know that we officially support lld for Apple platforms. People really should use the system linker there. (I get that some people might be trying to cross-compile to Apple platforms from Linux, which motivates them to use lld. That isn't something that is supported, and it might also be a violation of the licensing for the SDKs.)

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.

2 participants