Skip to content

Commit

Permalink
spack: update and fix dependencies for nvtx/roctracer
Browse files Browse the repository at this point in the history
- add dependency on `roctracer-dev` when(+rocm+nvtx)
- conflict +nvtx when neither rocm nor cuda is enabled
  • Loading branch information
simonpintarelli committed Dec 23, 2023
1 parent f87b315 commit 5ad61e6
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion spack/packages/sirius/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
"profiler", default=True, description="Use internal profiler to measure execution time"
)
variant(
"nvtx", default=False, description="Use NVTX profiler"
"nvtx", default=False, description="Use NVTX/ROCTX profiler"
)

depends_on("cmake@3.23:", type="build")
Expand Down Expand Up @@ -147,6 +147,7 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
depends_on("nlcglib+cuda", when="+nlcglib+cuda")

depends_on("libvdwxc@0.3.0:+mpi", when="+vdwxc")
depends_on("roctracer-dev", when="+nvtx+rocm")

depends_on("scalapack", when="+scalapack")

Expand All @@ -165,6 +166,7 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
conflicts("^libxc@5.0.0") # known to produce incorrect results
conflicts("+single_precision", when="@:7.2.4")
conflicts("+scalapack", when="^cray-libsci")
conflicts("+nvtx", when="~cuda~rocm")

# Propagate openmp to blas
depends_on("openblas threads=openmp", when="+openmp ^openblas")
Expand Down

0 comments on commit 5ad61e6

Please # to comment.