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

Compiler warnings when compiling core unit tests for Cuda #214

Closed
gmackey opened this issue Mar 16, 2016 · 3 comments
Closed

Compiler warnings when compiling core unit tests for Cuda #214

gmackey opened this issue Mar 16, 2016 · 3 comments
Assignees
Labels
Enhancement Improve existing capability; will potentially require voting
Milestone

Comments

@gmackey
Copy link
Contributor

gmackey commented Mar 16, 2016

When I compile for Cuda (7.5.18) in core/unit_test, I see a bunch of compiler warnings.

Here are the warnings:

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(130): warning: extern inline function "Kokkos::Impl::lock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(139): warning: extern inline function "Kokkos::Impl::unlock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(130): warning: extern inline function "Kokkos::Impl::lock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(139): warning: extern inline function "Kokkos::Impl::unlock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(130): warning: extern inline function "Kokkos::Impl::lock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(139): warning: extern inline function "Kokkos::Impl::unlock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(130): warning: extern inline function "Kokkos::Impl::lock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(139): warning: extern inline function "Kokkos::Impl::unlock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(130): warning: extern inline function "Kokkos::Impl::lock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Kokkos_Atomic.hpp(139): warning: extern inline function "Kokkos::Impl::unlock_address_cuda_space" was referenced but not defined

/home/gemacke/kokkos/clean_kokkos/core/src/Cuda/Kokkos_Cuda_Impl.cpp(106): warning: function "Kokkos::Impl::::cuda_launch_blocking" was declared but never referenced

/home/gemacke/kokkos/clean_kokkos/core/src/Cuda/Kokkos_Cuda_Impl.cpp(106): warning: function "Kokkos::Impl::::cuda_launch_blocking" was declared but never referenced

@gmackey gmackey added the Enhancement Improve existing capability; will potentially require voting label Mar 16, 2016
@hcedwar hcedwar added this to the Spring 2016 milestone Mar 17, 2016
@crtrott
Copy link
Member

crtrott commented Apr 28, 2016

The root cause is a forward declaration of an inline function. That inline function is forward declared, in Kokkos_Atomic, but only defined in Kokkos_CudaExec.hpp. Not sure yet how to resolve that, since both the forward declaration is necessary and I can't pull the functions forward (since they use a const symbol only declared in Kokkos_CudaExec.hpp, which I can't forward declare).

@crtrott
Copy link
Member

crtrott commented Apr 28, 2016

So I looked a bit more into this, and fixing it would involve a lot of cleanup and restructuring of internal code, in particular the organization of atomics. Considering other commitments for end of spring (in particular supporting execution space instances) this is not feasible, and I move this issue to backlog again.

@crtrott crtrott modified the milestones: Backlog, Spring 2016 Apr 28, 2016
@dsunder
Copy link
Contributor

dsunder commented Jun 13, 2017

Fixed by #881

@dsunder dsunder closed this as completed Jun 13, 2017
@dsunder dsunder reopened this Jun 13, 2017
@ibaned ibaned modified the milestones: Backlog, 2017-June-end Jun 14, 2017
@crtrott crtrott closed this as completed Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improve existing capability; will potentially require voting
Projects
None yet
Development

No branches or pull requests

5 participants