Daniel Arndt
Daniel Arndt
Fixes #7357. When Kokkos OpenMP parallel constructs are called from a OpenMP parallel region, we want to serialize the kernel. This also means that we should explicitly use resources dedicated...
We found that we have https://github.com/kokkos/kokkos/blob/638664a6d0263e4246097e34011279eee079fe21/cmake/KokkosConfigCommon.cmake.in#L6 which means we define `KOKKOS_CXX_COMPILER_ID` internally but export `Kokkos_CXX_COMPILER_ID` (and we don't define the former internally at all). This makes it awkward when guarding...
#7349 restricted tests if the number of detected threads is less than the numbers of partitions requested for an OpenMP execution space instance. However, its' not clear why `exec_space_thread_safety_team_policy` and...
Similar to https://github.com/kokkos/kokkos/pull/6496, this pull request introduces a new policy property called `RegisterFileSize` that allows forcing the register file size to small or large for Intel GPUs with the SYCL...
This requires https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/experimental/sycl_ext_oneapi_enqueue_functions.asciidoc. With the new syntax, we avoid creating backend events that we aren't using anyway when we are using in-order `sycl::queue`s.
We need to have the suppress before `Kokkos_Core.hpp` to be effective for the `Kokkos_NumcericTraits.hpp` and `Kokkos_Complex.hpp`. Also, enable warning and warnings as errors in the GitLab GH200 build.
This improved the compilation time of the tests from 30min to 5min. The runtime of the kernel is negligible.
```diff diff --git a/core/unit_test/TestViewOfViews.hpp b/core/unit_test/TestViewOfViews.hpp index 1d53bca33..2311fe93a 100644 --- a/core/unit_test/TestViewOfViews.hpp +++ b/core/unit_test/TestViewOfViews.hpp @@ -23,12 +23,12 @@ namespace { // User-defined types with a View data member template class S {...
This pull requedst makes the unit test results visible in the respective sections for the `jenkins` and `GItLab` CI.
#17442 made sure that we test with `Tpetra` classes with `MemorySpace::Default` but `MemorySpace::Host` is still the default. Should we change this default? From https://github.com/dealii/dealii/pull/17442#issue-2451522172: > In particular, using MemorySpace::Default allows...