Skip to content
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

Darwin: use libgcc_s version 1.1 on all targets #46240

Closed
wants to merge 1 commit into from

Conversation

fxcoudert
Copy link
Contributor

@fxcoudert fxcoudert commented Aug 2, 2022

The libgcc support shared library on modern GCC versions (11 and later) is called libgcc_s.1.1.dylib on all Darwin platforms. While a libgcc_s.1.dylib is still provided for backward compatibility on Intel Darwin, it is not recommended to use it.

On aarch64-darwin, libgcc_s.1.dylib is not provided at all, which is why this was special-cased in the build system. But actually, linking against libgcc_s.1.1.dylib in all cases on Darwin is the right way to go.

@fxcoudert fxcoudert mentioned this pull request Aug 2, 2022
6 tasks
@fxcoudert
Copy link
Contributor Author

Hum, is the compiler used in the pre-built CSL not recent enough?

@inkydragon
Copy link
Member

Apple clang version 11.0.3 (clang-1103.0.32.62)
Target: x86_64-apple-darwin21.4.0

https://buildkite.com/julialang/julia-master/builds/14479#01825f64-7e12-49b9-922d-584a2b10d7e0/321-324

@fxcoudert
Copy link
Contributor Author

OK, after checking this needs GCC 12. So this will need #45582 to be merged before re-testing.

@ViralBShah ViralBShah added the building Build system, or building Julia or its dependencies label Aug 4, 2022
@ViralBShah ViralBShah marked this pull request as draft September 5, 2022 21:31
@fxcoudert fxcoudert closed this Feb 27, 2023
@fxcoudert fxcoudert deleted the darwin branch February 27, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants