diff options
| author | Simon Pintarelli <1237199+simonpintarelli@users.noreply.github.com> | 2022-09-12 16:22:39 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-12 08:22:39 -0600 |
| commit | f33b7c0a58670117e8d64f38948d0bb3c8404630 (patch) | |
| tree | 892a889adb039fd3a7c2719587f02a39cc275a34 | |
| parent | d4065e11c6a6592d1ac8638cc4423695e37fbbde (diff) | |
| download | spack-f33b7c0a58670117e8d64f38948d0bb3c8404630.tar.gz spack-f33b7c0a58670117e8d64f38948d0bb3c8404630.tar.bz2 spack-f33b7c0a58670117e8d64f38948d0bb3c8404630.tar.xz spack-f33b7c0a58670117e8d64f38948d0bb3c8404630.zip | |
spfft: inherit from ROCmPackage (#32550)
Co-authored-by: Alberto Invernizzi <9337627+albestro@users.noreply.github.com>
| -rw-r--r-- | var/spack/repos/builtin/packages/spfft/package.py | 20 |
1 files changed, 1 insertions, 19 deletions
diff --git a/var/spack/repos/builtin/packages/spfft/package.py b/var/spack/repos/builtin/packages/spfft/package.py index de74d69333..2b516566a6 100644 --- a/var/spack/repos/builtin/packages/spfft/package.py +++ b/var/spack/repos/builtin/packages/spfft/package.py @@ -7,7 +7,7 @@ from spack.package import * -class Spfft(CMakePackage, CudaPackage): +class Spfft(CMakePackage, CudaPackage, ROCmPackage): """Sparse 3D FFT library with MPI, OpenMP, CUDA and ROCm support.""" homepage = "https://github.com/eth-cscs/SpFFT" @@ -49,9 +49,6 @@ class Spfft(CMakePackage, CudaPackage): depends_on("mpi", when="+mpi") depends_on("cmake@3.11:", type="build") - # ROCM variants + dependencies - variant("rocm", default=False, description="Use ROCm backend") - depends_on("cuda@:10", when="@:0.9.11 +cuda") with when("+rocm"): @@ -59,25 +56,10 @@ class Spfft(CMakePackage, CudaPackage): depends_on("hip@:4.0", when="@:1.0.1") # Workaround for compiler bug in ROCm 4.5 added in SpFFT 1.0.6 depends_on("hip@:4.3.1", when="@:1.0.5") - depends_on("hip") depends_on("rocfft") # rocFFT and hipFFT have split with latest versions depends_on("hipfft", when="^rocfft@4.1.0:") - amdgpu_targets = ( - "gfx701", - "gfx801", - "gfx802", - "gfx803", - "gfx900", - "gfx906", - "gfx908", - "gfx1010", - "gfx1011", - "gfx1012", - ) - variant("amdgpu_target", default="gfx803,gfx900,gfx906", multi=True, values=amdgpu_targets) - # Fix compilation error in some cases due to missing include statement # before version 1.0.3 patch("0001-fix-missing-limits-include.patch", when="@:1.0.2") |