Skip to content

Fix schedule_all move-only range support - #2203

Open
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix/schedule-all-move-only-range
Open

Fix schedule_all move-only range support#2203
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix/schedule-all-move-only-range

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

What changed

  • forward the range when building the schedule_all operation state
  • add a regression test with a move-only owning_view

Why

The consuming subscribe() path moves the range, but opstate_base_with_receiver passed its by-value range parameter as an lvalue. That made valid move-only ranges fail to compile.

Testing

  • build/test/exec/test.exec 'schedule_all on static_thread_pool accepts move-only ranges'
  • build/test/exec/test.exec

@copy-pr-bot

copy-pr-bot Bot commented Aug 15, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ericniebler

Copy link
Copy Markdown
Collaborator

/ok to test 43f0b4a

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.

2 participants