Skip to content

give up lambda-based mangling of __sexpr tparams - #2313

Merged
ericniebler merged 4 commits into
NVIDIA:mainfrom
ericniebler:remove-make-sexpr
Oct 9, 2026
Merged

ericniebler merged 4 commits into
NVIDIA:mainfrom
ericniebler:remove-make-sexpr

Conversation

@ericniebler

@ericniebler ericniebler commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

efforts to bring down the identifier length of sender types cause too many problems. parameterize __sexpr on the tag, data, and child senders instead of on a lambda that returns a descriptor.

this change results in longer sender identifiers. to ameliorate that issue, the just* senders get their own sender types that inherit from __sexpr instead of using __sexpr directly. likewise for the then and upon_(error|stopped) senders.

also, the __make_expr helper function causes an extra moves of the data and child parameters. rather than calling a function to make an __sexpr, just make the __sexpr directly with the help of CTAD. that way we benefit from deferred materialization of temporaries.

@copy-pr-bot

copy-pr-bot Bot commented Oct 8, 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 Author

/ok to test 6ff6ee9

@ericniebler

Copy link
Copy Markdown
Collaborator Author

/ok to test d8fa801

@ericniebler

Copy link
Copy Markdown
Collaborator Author

/ok to test bfe721f

@ericniebler

Copy link
Copy Markdown
Collaborator Author

/ok to test 6cc8b2f

@ericniebler

Copy link
Copy Markdown
Collaborator Author

/ok to test e80205f

@ericniebler ericniebler changed the title remove unnecessary __make_sexpr helper give up lambda-based mangling of __sexpr tparams Oct 9, 2026
@ericniebler
ericniebler merged commit d76067b into NVIDIA:main Oct 9, 2026
69 of 70 checks passed
@ericniebler
ericniebler deleted the remove-make-sexpr branch October 9, 2026 05:48
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
romintomasetti added a commit to uliegecsm/kokkos-execution that referenced this pull request Oct 9, 2026
_WITH_PRETTY_SENDER_ disappeared in NVIDIA/stdexec#2313 (as well as the attempt to mangle/reduce the length of the sender types)

Signed-off-by: romintomasetti <romin.tomasetti@gmail.com>
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.

1 participant