Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Eliminate comma operator for placeholders #6

Closed
jaredhoberock opened this issue Mar 15, 2012 · 0 comments · Fixed by #111
Closed

Eliminate comma operator for placeholders #6

jaredhoberock opened this issue Mar 15, 2012 · 0 comments · Fixed by #111
Labels
type: enhancement New feature or request.

Comments

@jaredhoberock
Copy link
Contributor

It causes problems in expressions like

thrust::transform_reduce(v.begin(), v.end(), _1 * _1, 0.0f, _1 + _2);

I don't know how to disambiguate cases like these and I can't think of any reason to keep it around.
@jaredhoberock jaredhoberock mentioned this issue May 23, 2012
Merged
jaredhoberock added a commit to jaredhoberock/thrust that referenced this issue May 31, 2012
@jaredhoberock
Merge pull request NVIDIA#111 from jaredhoberock/integrate-issue-6
a5e6dc2 
Eliminate the comma operator for placeholder expressions.

Fixes NVIDIA#6
kshitij12345 pushed a commit to kshitij12345/thrust that referenced this issue Mar 24, 2022
@dumerrill
Merge pull request NVIDIA#6 from NVlabs/master
29d354f 
Max and linux fixes

Former-commit-id: 561322b1a3c6e146d68f40dfd3708b9cecb180c4
kshitij12345 pushed a commit to kshitij12345/thrust that referenced this issue Mar 24, 2022
@dumerrill
Merge pull request NVIDIA#6 from egaburov/cub-fixes
aa6ea7c 
http://nvbugs/1904217
Former-commit-id: f5c46e7272b9e33b3c6bbdf9145a0e43dc47ea39
@LinGeLin LinGeLin mentioned this issue Oct 9, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
type: enhancement New feature or request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Eliminate the comma operator for placeholder expressions. jaredhoberock/thrust
1 participant
@jaredhoberock