Skip to content

Commit

Permalink
Merge pull request #3500 from vicentebolea/remove-kokkos-cuda-warnings
Browse files Browse the repository at this point in the history
ci,kokkos,cuda: remove warnings
  • Loading branch information
vicentebolea authored Feb 22, 2023
2 parents 2c587e6 + 0ee5821 commit 5986945
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ adios_option(Catalyst "Enable support for in situ visualization plugin using P
adios_option(AWSSDK "Enable support for S3 compatible storage using AWS SDK's S3 module" AUTO)
include(${PROJECT_SOURCE_DIR}/cmake/DetectOptions.cmake)

if(ADIOS2_HAVE_CUDA)
if(ADIOS2_HAVE_CUDA OR ADIOS2_HAVE_Kokkos_CUDA)
set(CMAKE_CUDA_STANDARD 11)
set(CMAKE_CUDA_STANDARD_REQUIRED TRUE)
if(NOT DEFINED CMAKE_CUDA_ARCHITECTURES)
Expand Down
2 changes: 1 addition & 1 deletion docs/user_guide/source/advanced/gpu_aware.rst
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ When building ADIOS2 with CUDA enabled, the user is responsible with setting the
Building with Kokkos enabled
--------------------------

The Kokkos library can be used to enable GPU within ADIOS2. Based on how Kokkos is build, either the CUDA or HIP backend will be enabled. Building with Kokkos requires ``-DADIOS2_USE_Kokkos=ON``. The user is not responsible to set the ``CMAKE_CUDA_ARCHITECTURES`` since it will be detected from the Kokkos installation.
The Kokkos library can be used to enable GPU within ADIOS2. Based on how Kokkos is build, either the CUDA or HIP backend will be enabled. Building with Kokkos requires ``-DADIOS2_USE_Kokkos=ON``. The user is responsible to set the ``CMAKE_CUDA_ARCHITECTURES`` to the same architecture used when configuring the Kokkos library it links against.

.. note::
Kokkos version >= 3.7 is required to enable the GPU backend in ADIOS2
Expand Down

0 comments on commit 5986945

Please sign in to comment.