zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Allow Kokkos with OpenMPTarget backend (#30724)

Browse Source 
* Allow Kokkos with OpenMPTarget backend

* Restrict SYCL and OpenMPTarget to C++17 or higher

* Improve C++ standard check for SYCL and OpenMPTarget

* Fix indentation
This commit is contained in:
Daniel Arndt 2022-05-24 17:23:47 -04:00 committed by GitHub
parent 067800bc31
commit 54ea1f4bf6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
var/spack/repos/builtin/packages/kokkos/package.py
View File

@ -46,9 +46,11 @@ class Kokkos(CMakePackage, CudaPackage, ROCmPackage):
'serial': [True, 'Whether to build serial backend'], 'serial': [True, 'Whether to build serial backend'],
'rocm': [False, 'Whether to build HIP backend'], 'rocm': [False, 'Whether to build HIP backend'],
'sycl': [False, 'Whether to build the SYCL backend'], 'sycl': [False, 'Whether to build the SYCL backend'],
'openmptarget': [False, 'Whether to build the OpenMPTarget backend']
} }
conflicts("+rocm", when="@:3.0") conflicts("+rocm", when="@:3.0")
conflicts("+sycl", when="@:3.3") conflicts("+sycl", when="@:3.3")
conflicts("+openmptarget", when="@:3.5")
# https://github.com/spack/spack/issues/29052 # https://github.com/spack/spack/issues/29052
conflicts("@:3.5.00 +sycl", when="%dpcpp@2022.0.0") conflicts("@:3.5.00 +sycl", when="%dpcpp@2022.0.0")
@ -194,6 +196,13 @@ class Kokkos(CMakePackage, CudaPackage, ROCmPackage):
conflicts("+cuda", when="std=17 ^cuda@:10") conflicts("+cuda", when="std=17 ^cuda@:10")
conflicts("+cuda", when="std=20") conflicts("+cuda", when="std=20")
# SYCL and OpenMPTarget require C++17 or higher
for stdver in stds[:stds.index('17')]:
conflicts('+sycl', when='std={0}'.format(stdver),
msg='SYCL requires C++17 or higher')
conflicts("+openmptarget", when='std={0}'.format(stdver),
msg='OpenMPTarget requires C++17 or higher')
# HPX should use the same C++ standard # HPX should use the same C++ standard
for std in stds: for std in stds:
depends_on('hpx cxxstd={0}'.format(std), when='+hpx std={0}'.format(std)) depends_on('hpx cxxstd={0}'.format(std), when='+hpx std={0}'.format(std))