Skip to content
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

Allow kernel argument to take custom struct by value #834

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rosenrodt
Copy link
Contributor

By specializing template<T> struct set_kernel_arg when expanding BOOST_COMPUTE_ADAPT_STRUCT, kernel input argument can now take custom struct by value. This potentially utilizes constant memory of the compute device

@rosenrodt rosenrodt force-pushed the pr-kernel-pass-struct-by-value branch from 11401ed to b040362 Compare May 9, 2019 13:37
@rosenrodt rosenrodt force-pushed the pr-kernel-pass-struct-by-value branch from b040362 to e1b96b3 Compare May 9, 2019 14:34
@coveralls
Copy link

coveralls commented May 9, 2019

Coverage Status

Coverage remained the same at 84.019% when pulling e1b96b3 on rosenrodt:pr-kernel-pass-struct-by-value into 36c8913 on boostorg:master.

@rosenrodt
Copy link
Contributor Author

It looks like Apple platform regards custom struct as invalid kernel argument. That's weird because my 2016 Macbook works just fine

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.

None yet

2 participants