Skip to content

[2026-03 LWG Motion 35] P3927R2 task_scheduler support for parallel bulk execution#8927

Merged
tkoeppe merged 1 commit intomainfrom
motions-2026-03-lwg-35
Apr 20, 2026
Merged

[2026-03 LWG Motion 35] P3927R2 task_scheduler support for parallel bulk execution#8927
tkoeppe merged 1 commit intomainfrom
motions-2026-03-lwg-35

Conversation

@jensmaurer
Copy link
Copy Markdown
Member

Fixes NB US 238-368 (C++26 CD).

Fixes #8869

Also fixes cplusplus/papers#2600
Also fixes cplusplus/nbballot#943

@jensmaurer jensmaurer added this to the post-2026-03 milestone Apr 7, 2026
@jensmaurer jensmaurer force-pushed the motions-2026-03-lwg-35 branch 2 times, most recently from 2065583 to dbbd056 Compare April 7, 2026 12:48
Copy link
Copy Markdown
Contributor

@ericniebler ericniebler left a comment

Choose a reason for hiding this comment

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

looks good. thanks!

Comment thread source/exec.tex
Comment thread source/exec.tex Outdated
@jensmaurer jensmaurer force-pushed the motions-2026-03-lwg-35 branch 2 times, most recently from 445b9de to 301d230 Compare April 10, 2026 09:39
@tkoeppe tkoeppe force-pushed the motions-2026-03-lwg-35 branch from 301d230 to 6e6b097 Compare April 20, 2026 12:30
Comment thread source/exec.tex Outdated
Comment thread source/exec.tex
Comment thread source/exec.tex
Copy link
Copy Markdown
Member

@jwakely jwakely left a comment

Choose a reason for hiding this comment

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

There's a stray x in the pull request.

There are two places where the paper says that ts-domain::transform_sender is exposition-only, which are not in the pull request. But if it's exposition-only, why isn't it called transform-sender instead? If its name needs to be transform_sender then it's not exposition only, and the pull request is correct. I don't know.

@tkoeppe tkoeppe force-pushed the motions-2026-03-lwg-35 branch from 6e6b097 to 8a93fc6 Compare April 20, 2026 13:08
Fixes NB US 238-368 (C++26 CD).

Editorial note:
* [exec.task.scheduler] Change "Initialize sch_ with" to "Initializes
  sch_ with:", and put the (long) expression in a code block.
@tkoeppe tkoeppe force-pushed the motions-2026-03-lwg-35 branch from 8a93fc6 to 4956880 Compare April 20, 2026 13:47
@tkoeppe tkoeppe requested a review from jwakely April 20, 2026 13:47
@tkoeppe
Copy link
Copy Markdown
Contributor

tkoeppe commented Apr 20, 2026

There are two places where the paper says that ts-domain::transform_sender is exposition-only, which are not in the pull request. But if it's exposition-only, why isn't it called transform-sender instead? If its name needs to be transform_sender then it's not exposition only, and the pull request is correct. I don't know.

Since ts-domain itself is an exposition-only class, I don't think it's observable whether its members are also exposition-only, and we don't need to also annotate members. E.g. the members of backend-for (added in the same paper) are also not annotated. I think that's fine, I don't see what else it could mean.

@tkoeppe tkoeppe merged commit 9beabb4 into main Apr 20, 2026
3 of 4 checks passed
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.

[2026-03 LWG Motion 35] P3927R2 task_scheduler support for parallel bulk execution P3927 R0 task_scheduler Support for Parallel Bulk Execution

4 participants