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 Feb 6, 2024
1 parent 13d64e5 commit 9a76008
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion spack/packages/sirius/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,9 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
variant(
"profiler", default=True, description="Use internal profiler to measure execution time"
)
variant("nvtx", default=False, description="Use NVTX profiler")
variant(
"nvtx", default=False, description="Use NVTX/ROCTX profiler"
)

depends_on("[email protected]:", type="build")
depends_on("mpi")
Expand Down Expand Up @@ -150,6 +152,7 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
depends_on("nlcglib+cuda", when="+nlcglib+cuda")

depends_on("[email protected]:+mpi", when="+vdwxc")
depends_on("roctracer-dev", when="+nvtx+rocm")

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

Expand All @@ -168,6 +171,7 @@ class Sirius(CMakePackage, CudaPackage, ROCmPackage):
conflicts("^[email protected]") # 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 9a76008

Please sign in to comment.