-
Notifications
You must be signed in to change notification settings - Fork 407
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
Introduce SYCLHostUSMSpace #4268
Conversation
527e771
to
4845630
Compare
4845630
to
6fe6962
Compare
static_assert( | ||
std::is_same< | ||
Kokkos::Device<Kokkos::HostSpace::execution_space, | ||
Kokkos::Experimental::SYCLSharedUSMSpace>, | ||
Kokkos::Device<Kokkos::HostSpace::execution_space, | ||
Kokkos::Experimental::SYCLSharedUSMSpace>>::value, | ||
""); |
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 see that this just copied from Cuda
kokkos/core/unit_test/cuda/TestCuda_Spaces.cpp
Lines 228 to 232 in 422c820
static_assert(std::is_same<Kokkos::Device<Kokkos::HostSpace::execution_space, | |
Kokkos::CudaUVMSpace>, | |
Kokkos::Device<Kokkos::HostSpace::execution_space, | |
Kokkos::CudaUVMSpace>>::value, | |
""); |
but can't help noting it is pointless.
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.
Almost there
Retest this please. |
@@ -146,6 +183,11 @@ static_assert(Kokkos::Impl::MemorySpaceAccess< | |||
Kokkos::Experimental::SYCLSharedUSMSpace>::assignable, | |||
""); | |||
|
|||
static_assert(Kokkos::Impl::MemorySpaceAccess< | |||
Kokkos::Experimental::SYCLDeviceUSMSpace, | |||
Kokkos::Experimental::SYCLDeviceUSMSpace>::assignable, |
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.
Shouldn't these be Kokkos::Experimental::SYCLHostUSMSpace
?
Depends on this PR from Kokkos: kokkos/kokkos#4268
We were discussing that there are situations where
SYCLHostUSMSpace
would have advantages overSYCLSharedUSMSpace
andSYCLDeviceUSMSpace
(like launch latencies). To allow exploring this some more and to make the backend more similar to theCUDA
one, this pull request introduces a new memory space.Additionally, this pull request copies the relevant backend-specific tests from the
CUDA
tests.