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

vecmem: add infrastructure for working with SYCL (#45058)

Browse Source 
* vecmem: add infrastructure for working with SYCL

The vecmem package uses an unorthodox build system where, instead of
expecting a SYCL-capable compiler in the `CXX` environment variable, it
expects one in `SYCLCXX`. It also needs the correct SYCL flags to be
set. This commit adds a custom build environment for the vecmem package
which allows it to be built in this way. I've also added an extra CMake
flag to ensure that the build system doesn't download any unwanted
dependencies.

* Update var/spack/repos/builtin/packages/vecmem/package.py

Co-authored-by: Wouter Deconinck <wdconinc@gmail.com>

---------

Co-authored-by: Wouter Deconinck <wdconinc@gmail.com>
This commit is contained in:
Stephen Nicholas Swatman 2024-07-15 21:42:19 +02:00 committed by GitHub
parent a26ac1dbcc
commit c12772e73f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
var/spack/repos/builtin/packages/vecmem/package.py
View File

@ -63,6 +63,12 @@ class Vecmem(CMakePackage, CudaPackage):
# and we can choose between always depending on googletest, or using FetchContent
# depends_on("googletest", type="test")
def setup_build_environment(self, env):
if self.spec.satisfies("+sycl"):
env.set("SYCLCXX", self.compiler.cxx)
if self.spec.satisfies("%oneapi"):
env.set("SYCLFLAGS", "-fsycl")
def cmake_args(self):
args = [
self.define("FETCHCONTENT_FULLY_DISCONNECTED", False), # see FIXME above
@ -71,6 +77,7 @@ def cmake_args(self):
self.define_from_variant("VECMEM_BUILD_SYCL_LIBRARY", "sycl"),
self.define("BUILD_TESTING", self.run_tests),
self.define("VECMEM_BUILD_TESTING", self.run_tests),
self.define("VECMEM_USE_SYSTEM_LIBS", True),
self.define("VECMEM_USE_SYSTEM_GOOGLETEST", False), # see FIXME above
]