Skip to content

Link to libgcc for armv5te-unknown-linux-musleabi target. #987

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
Aug 10, 2022

Conversation

Alexhuszagh
Copy link
Contributor

Fixes missing sync builtins required for libraries such as OpenSSL, specifically, the __sync_X_and_fetch builtins.

Closes #367.
Improves documentation related to #906.

@Alexhuszagh Alexhuszagh added bug upstream A-musl Area: musl libc targets labels Jul 30, 2022
@Alexhuszagh Alexhuszagh requested a review from a team as a code owner July 30, 2022 16:12
@Emilgardis
Copy link
Member

Emilgardis commented Jul 30, 2022

I'm not sure this is something we should do since it's an upstream issue and will eventually be fixed hopefully.

@Alexhuszagh
Copy link
Contributor Author

Alexhuszagh commented Jul 30, 2022

I'm not sure this is something we should do since it's an upstream issue and will eventually be fixed hopefully.

It is, it's already been patched (I submitted rust-lang/compiler-builtins#484 which was merged), but we need to wait for a stable release similar to what we do in aarch64-linux-musl-gcc.sh Similarly, we can remove -libgcc shortly when rust-lang/compiler-builtins#483 makes it to stable.

Copy link
Member

@Emilgardis Emilgardis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's merge this, we can add a version check later maybe?

r=me if ok

@Alexhuszagh
Copy link
Contributor Author

bors r=Emilgardis

bors bot added a commit that referenced this pull request Aug 10, 2022
987: Link to libgcc for `armv5te-unknown-linux-musleabi` target. r=Emilgardis a=Alexhuszagh

Fixes missing sync builtins required for libraries such as OpenSSL, specifically, the `__sync_X_and_fetch` builtins.

Closes #367.
Improves documentation related to #906.

Co-authored-by: Alex Huszagh <[email protected]>
@bors
Copy link
Contributor

bors bot commented Aug 10, 2022

Build failed:

Fixes missing sync builtins required for libraries such as OpenSSL,
specifically, the `__sync_X_and_fetch` builtins.
@Alexhuszagh Alexhuszagh force-pushed the armv5te_musl_builtins branch from 252bda3 to 44184ec Compare August 10, 2022 17:48
@Alexhuszagh
Copy link
Contributor Author

bors try --target mips64-unknown-linux-muslabi64

bors bot added a commit that referenced this pull request Aug 10, 2022
@bors
Copy link
Contributor

bors bot commented Aug 10, 2022

try

Build succeeded:

@Alexhuszagh
Copy link
Contributor Author

bors r=Emilgardis

@bors
Copy link
Contributor

bors bot commented Aug 10, 2022

Build succeeded:

@bors bors bot merged commit a93c2b7 into cross-rs:main Aug 10, 2022
@Emilgardis Emilgardis added this to the v0.3.0 milestone Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-musl Area: musl libc targets bug upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

armv5te-unknown-linux-musleabi with OpenSSL fails with undefined reference to `__sync_sub_and_fetch_4'
2 participants