Skip to content

Add support for -no-pthread - #20723

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:no_pthread
Nov 15, 2023
Merged

Add support for -no-pthread#20723
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:no_pthread

Conversation

@sbc100

@sbc100 sbc100 commented Nov 15, 2023

Copy link
Copy Markdown
Collaborator

Fixes: #20720

@sbc100
sbc100 requested review from RReverser and kripken November 15, 2023 18:20
@sbc100
sbc100 enabled auto-merge (squash) November 15, 2023 18:34

@RReverser RReverser left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Wow, thanks, this was quick!

@sbc100
sbc100 merged commit 5b6bf91 into emscripten-core:main Nov 15, 2023
@sbc100
sbc100 deleted the no_pthread branch November 15, 2023 19:29
@sbc100

sbc100 commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator Author

#27546

dschuff pushed a commit that referenced this pull request Aug 17, 2026
It turns out this was never really supported by LLVM in the way I
thought it was when I added support here:
llvm/llvm-project#216378
  
I'm pretty we never advertised the availability of this non-standard
flag anywhere so I think removing it should be low risk. See #20723 for
where it was added.
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.

Support -no-pthread linking flag

3 participants