Skip to content

exec::repeat_until et al.: Dependent Child Sender Support#1879

Merged
ericniebler merged 8 commits intoNVIDIA:mainfrom
RobertLeahy:repeat_until_20200221
Mar 3, 2026
Merged

exec::repeat_until et al.: Dependent Child Sender Support#1879
ericniebler merged 8 commits intoNVIDIA:mainfrom
RobertLeahy:repeat_until_20200221

Conversation

@RobertLeahy
Copy link
Contributor

@RobertLeahy RobertLeahy commented Feb 21, 2026

The TODO eliminated by this commit not only meant that exec::repeat_until et al. previously didn't the usual constant evaluation techniques to report errors, it also meant that a hard compile error (rather than a return type indicating that the sender is dependent) resulted if:

  • The child sender was dependent, and
  • An attempt was made to determine the completion signatures of the parent without an environment

Updated so exec::repeat_until et al. now properly report that they are dependent if their child is dependent.

The TODO eliminated by this commit not only meant that exec::
repeat_until et al. previously didn't the usual constant evaluation
techniques to report errors, it also meant that a hard compile error
(rather than a return type indicating that the sender is dependent)
resulted if:

- The child sender was dependent, and
- An attempt was made to determine the completion signatures of the
  parent without an environment

Updated so exec::repeat_until et al. now properly report that they are
dependent if their child is dependent.
@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 21, 2026

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
Collaborator

/ok to test 62c7d22

@ericniebler
Copy link
Collaborator

/ok to test 3d97de3

@ericniebler
Copy link
Collaborator

/ok to test 03d29fd

@ericniebler
Copy link
Collaborator

/ok to test 299ba24

@ericniebler
Copy link
Collaborator

/ok to test 5eda5dc

@ericniebler ericniebler merged commit 7815a8f into NVIDIA:main Mar 3, 2026
26 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.

2 participants