Skip to content

[SYCL] Define collective functors using std #840

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 19, 2019
Merged

Conversation

Pennycook
Copy link
Contributor

intel::plus should be an alias to std::plus
intel::minimum/maximum should use std::less/std::greater

Using the <void> specializations is now only possible using C++14.

intel::plus should be an alias to std::plus
intel::minimum/maximum should use std::less/std::greater

Signed-off-by: John Pennycook <[email protected]>
plus<>, minimum<> and maximum<> depend on C++14.

Signed-off-by: John Pennycook <[email protected]>
void specializations should be transparent, accepting arbitrary types
and using perfect forwarding.

Signed-off-by: John Pennycook <[email protected]>
@bader
Copy link
Contributor

bader commented Nov 19, 2019

LGTM

@bader bader merged commit d35a815 into intel:sycl Nov 19, 2019
@Pennycook Pennycook deleted the functional branch March 3, 2020 15:39
vmaksimo pushed a commit to vmaksimo/llvm that referenced this pull request May 18, 2021
)

Expose SPIRVToOCL12, SPIRVToOCL20 interfaces in SPIRVToOCL.h

This "recommit"s intel#904, which was accidentally "revert"ed by intel#840

Signed-off-by: Yilong Guo <[email protected]>

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@6171e51
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.

3 participants