Skip to content

Commit 9d66794

Browse files
authored
Merge pull request #152 from cwpearson/fix/mpi-ext
2 parents cbe84df + ca29658 commit 9d66794

File tree

1 file changed

+13
-5
lines changed

1 file changed

+13
-5
lines changed

unit_tests/mpi/test_mpi_view_access.cpp

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,17 +31,25 @@ using namespace KokkosComm::mpi;
3131
namespace {
3232

3333
void view_access_via_mpi_extension() {
34-
#ifdef KOKKOSCOMM_IMPL_MPIEXT_H
3534
#if defined(KOKKOSCOMM_IMPL_MPI_IS_OPENMPI) && defined(KOKKOS_ENABLE_CUDA)
35+
36+
#if defined(KOKKOSCOMM_IMPL_MPIEXT_H)
3637
EXPECT_EQ(1, MPIX_Query_cuda_support());
37-
#elif defined(KOKKOSCOMM_IMPL_MPI_IS_MPICH) && defined(KOKKOS_ENABLE_CUDA)
38+
#else
39+
GTEST_SKIP() << "mpi-ext.h not available";
40+
#endif
41+
42+
#elif defined(KOKKOSCOMM_IMPL_MPI_IS_MPICH)
43+
44+
#if defined(KOKKOS_ENABLE_CUDA)
3845
EXPECT_EQ(1, MPIX_Query_cuda_support());
3946
#else
40-
GTEST_SKIP() << "Query CUDA support via MPI extension not available";
47+
GTEST_SKIP() << "CUDA not enabled";
4148
#endif
49+
4250
#else
43-
GTEST_SKIP() << "mpi-ext.h not available";
44-
#endif // KOKKOSCOMM_IMPL_MPI_EXT_H
51+
GTEST_SKIP() << "unable to check for CUDA-aware MPI";
52+
#endif
4553
}
4654

4755
void doit() {

0 commit comments

Comments
 (0)