Skip to content

Commit 8047a39

Browse files
Fix missing MpiSpace
Signed-off-by: Cedric Chevalier <[email protected]>
1 parent d49bffc commit 8047a39

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

src/KokkosComm/mpi/allgather.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
namespace KokkosComm::mpi {
1818

1919
template <KokkosExecutionSpace ExecSpace, KokkosView SView, KokkosView RView>
20-
auto iallgather(const ExecSpace &space, const SView sv, RView rv, MPI_Comm comm) -> Req<Mpi> {
20+
auto iallgather(const ExecSpace &space, const SView sv, RView rv, MPI_Comm comm) -> Req<MpiSpace> {
2121
using ST = typename SView::non_const_value_type;
2222
using RT = typename RView::non_const_value_type;
2323
static_assert(std::is_same_v<ST, RT>, "KokkosComm::mpi::iallgather: View value types must be identical");
@@ -29,7 +29,7 @@ auto iallgather(const ExecSpace &space, const SView sv, RView rv, MPI_Comm comm)
2929
// Sync: Work in space may have been used to produce view data.
3030
space.fence("fence before non-blocking all-gather");
3131

32-
Req<Mpi> req;
32+
Req<MpiSpace> req;
3333
// All ranks send/recv same count
3434
MPI_Iallgather(data_handle(sv), span(sv), Impl::mpi_type_v<ST>, data_handle(rv), span(sv), Impl::mpi_type_v<RT>, comm,
3535
&req.mpi_request());

src/KokkosComm/mpi/alltoall.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
namespace KokkosComm::mpi {
1919

2020
template <KokkosExecutionSpace ExecSpace, KokkosView SView, KokkosView RView>
21-
auto ialltoall(const ExecSpace &space, const SView sv, RView rv, int count, MPI_Comm comm) -> Req<Mpi> {
21+
auto ialltoall(const ExecSpace &space, const SView sv, RView rv, int count, MPI_Comm comm) -> Req<MpiSpace> {
2222
using ST = typename SView::non_const_value_type;
2323
using RT = typename RView::non_const_value_type;
2424
static_assert(std::is_same_v<ST, RT>, "KokkosComm::mpi::ialltoall: View value types must be identical");
@@ -30,7 +30,7 @@ auto ialltoall(const ExecSpace &space, const SView sv, RView rv, int count, MPI_
3030
// Sync: Work in space may have been used to produce view data.
3131
space.fence("fence before non-blocking all-gather");
3232

33-
Req<Mpi> req;
33+
Req<MpiSpace> req;
3434
// All ranks send/recv same count
3535
MPI_Ialltoall(data_handle(sv), count, Impl::mpi_type_v<ST>, data_handle(rv), count, Impl::mpi_type_v<RT>, comm,
3636
&req.mpi_request());

src/KokkosComm/mpi/broadcast.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@
1717
namespace KokkosComm::mpi {
1818

1919
template <KokkosExecutionSpace ExecSpace, KokkosView View>
20-
auto ibroadcast(const ExecSpace& space, View& v, int root, MPI_Comm comm) -> Req<Mpi> {
20+
auto ibroadcast(const ExecSpace& space, View& v, int root, MPI_Comm comm) -> Req<MpiSpace> {
2121
using T = typename View::non_const_value_type;
2222
Kokkos::Tools::pushRegion("KokkosComm::mpi::ibroadcast");
2323
fail_if(!is_contiguous(v), "KokkosComm::mpi::ibroadcast: unimplemented for non-contiguous views");
2424

2525
// Sync: Work in space may have been used to produce view data.
2626
space.fence("fence before non-blocking broadcast");
2727

28-
Req<Mpi> req;
28+
Req<MpiSpace> req;
2929
MPI_Ibcast(data_handle(v), span(v), Impl::mpi_type_v<T>, root, comm, &req.mpi_request());
3030
req.extend_view_lifetime(v);
3131

0 commit comments

Comments
 (0)