Skip to content

Add check if libgcc is available and link winpthreads against it instead of fakelib - #7

Closed
Blackhex wants to merge 1 commit into
woarm64from
fix-stage2-fakelib
Closed

Add check if libgcc is available and link winpthreads against it instead of fakelib#7
Blackhex wants to merge 1 commit into
woarm64from
fix-stage2-fakelib

Conversation

@Blackhex

@Blackhex Blackhex commented Mar 19, 2025

Copy link
Copy Markdown
Member

winpthreads is linking against fake runtime library for stage 1 builds. During stage 2 when libgcc is already available, this i not needed.

This PR fixes Windows-on-ARM-Experiments/mingw-woarm64-build#208 and Windows-on-ARM-Experiments/mingw-woarm64-build#205.

Validated by https://github.com/Windows-on-ARM-Experiments/mingw-woarm64-build/actions/runs/13951078487

@Blackhex
Blackhex marked this pull request as draft March 19, 2025 15:45
@Blackhex
Blackhex force-pushed the fix-stage2-fakelib branch 3 times, most recently from c6e9c39 to ba6b770 Compare March 19, 2025 15:55

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This file is updated automatically by automake.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This file is updated automatically by autoconf.

@Blackhex Blackhex changed the title Add check if libgcc is available and link winpthreads against it instead of fakelib WIP: Add check if libgcc is available and link winpthreads against it instead of fakelib Mar 20, 2025
@Blackhex
Blackhex force-pushed the fix-stage2-fakelib branch from ba6b770 to abc2a28 Compare March 20, 2025 13:41
@Blackhex Blackhex changed the title WIP: Add check if libgcc is available and link winpthreads against it instead of fakelib Add check if libgcc is available and link winpthreads against it instead of fakelib Mar 21, 2025
@Blackhex
Blackhex marked this pull request as ready for review March 22, 2025 21:00
@Blackhex Blackhex self-assigned this Mar 22, 2025
@Blackhex
Blackhex requested review from eukarpov, thiru-mcw and vejbomar and removed request for eukarpov, thiru-mcw and vejbomar March 22, 2025 21:01
@Blackhex

Copy link
Copy Markdown
Member Author

This PR solves the same issue as #8 in different way. After discussions with MinGW community the other one seems like the preferred approach. So, closing this one.

@Blackhex Blackhex closed this May 26, 2025
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.

MinGW: libwinpthreads and crt need to be built with -mno-outline-atomics

1 participant