-
Notifications
You must be signed in to change notification settings - Fork 312
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
[SofaCUDA,SofaDistanceGrid] Move DistanceGrid CUDA files in a SofaDistanceGrid extension #4878
Conversation
ea41c6f
to
17a804c
Compare
17a804c
to
4ee5f02
Compare
|
||
|
||
namespace sofa::gpu::cuda | ||
#if defined(__cplusplus) && CUDA_VERSION < 2000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also would like to know. NVCC is still a mystery for me...
Compilation issue on Windows. Modernize the CUDA Cmake before this PR. This plugin adds C source file, making it harder than other reordering done before. Problem arise from the fact that CMake passes compile option to NVCC that the compiler doesn't understand. |
aa5e7b0
to
2930512
Compare
2930512
to
c0dad15
Compare
[ci-build][with-all-tests] |
cccf508
to
4425187
Compare
[ci-build][with-all-tests][force-full-build] |
1 similar comment
[ci-build][with-all-tests][force-full-build] |
4425187
to
78151f1
Compare
78151f1
to
73a67ae
Compare
[ci-build][with-all-tests][force-full-build] |
73a67ae
to
bf49423
Compare
[ci-build][with-all-tests][force-full-build] |
By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).
Reviewers will merge this pull-request only if