Skip to content

[doc] PTHREAD_POOL_SIZE supports JS expressions#14251

Merged
kripken merged 1 commit intomainfrom
document-pthread-pool-size-expr
May 25, 2021
Merged

[doc] PTHREAD_POOL_SIZE supports JS expressions#14251
kripken merged 1 commit intomainfrom
document-pthread-pool-size-expr

Conversation

@RReverser
Copy link
Copy Markdown
Collaborator

Fixes #12302.

@RReverser RReverser requested a review from kripken May 21, 2021 12:38
Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

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

Thanks!

@RReverser
Copy link
Copy Markdown
Collaborator Author

RReverser commented May 21, 2021

@kripken Actually, I just remembered that someone suggested _emscripten_num_logical_cores on the original issue instead: #12302 (comment)

I guess we should recommend it instead? If so, should it be =_emscripten_num_logical_cores or =Module.emscripten_num_logical_cores?

@kripken
Copy link
Copy Markdown
Member

kripken commented May 22, 2021

I think the PR is good as it is. To use a C method, it would need to be exported, for one thing, which adds complexity to the example. Using a natural JS example in this PR seems best.

@RReverser RReverser enabled auto-merge (squash) May 25, 2021 11:53
@RReverser
Copy link
Copy Markdown
Collaborator Author

Can't merge it due to the same Firefox CI issues :(

@kripken
Copy link
Copy Markdown
Member

kripken commented May 25, 2021

Sorry about that, I'll merge it now.

@kripken kripken disabled auto-merge May 25, 2021 15:46
@kripken kripken merged commit 4b9c93c into main May 25, 2021
@kripken kripken deleted the document-pthread-pool-size-expr branch May 25, 2021 15:46
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.

Document that PTHREAD_POOL_SIZE accepts JS expressions

2 participants