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

Cublas demo #2461

Merged
merged 2 commits into from Jan 24, 2017
Merged

Cublas demo #2461

merged 2 commits into from Jan 24, 2017

Conversation

biddisco
Copy link
Contributor

Demonstrates how to mix calls to CUDA cublas with hpx futures

@hkaiser
Copy link
Member

hkaiser commented Jan 20, 2017

@biddisco Thanks a lot! Does this now work with Boost V1.63 as well?

Also, could you please configure clang-format such that it does not align assignments and variable declarations?

@biddisco
Copy link
Contributor Author

I added an #ifdef xxx_CUDA_ALLOCATOR around all the code that used the cuda::allocator and replaced the allocation and parallel::copy with cudaMalloc and cudaMemcpy - this means we can use the c++ compiler and everything compiles fine with any boost etc. When using the allocator, I was not able to get the nvcc compiler to work and did not want to remove all lambdas, so currently it will only compile (for me) using the non allocator version. My collaborators assure me that they will never use the allocators/copy/compute/etc that hpx provides and just need a demo that shows how to mix cublas and futures.

@biddisco
Copy link
Contributor Author

I "align assignments and variable declarations" by hand because I like it. Are you telling me to stop?

@hkaiser
Copy link
Member

hkaiser commented Jan 21, 2017

I "align assignments and variable declarations" by hand because I like it. Are you telling me to stop?

We don't align variables or assignments anywhere else. But we don't enforce any particular style, only encourage authors not to change formatting in existing files. We also encourage authors to keep formatting of changed lines in existing files consistent with the existing style.

@biddisco
Copy link
Contributor Author

I have put the indentations/alignments I changed back the way they were.

Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@biddisco biddisco merged commit 1f3f324 into master Jan 24, 2017
@biddisco biddisco deleted the cublas_demo branch January 24, 2017 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants