kokkos / kokkos-comm

Experimental MPI Wrapper for Kokkos
https://kokkos.org/kokkos-comm/
Other
16 stars 10 forks source link

Add fences for alltoall #84

Closed cedricchevalier19 closed 5 months ago

cedricchevalier19 commented 5 months ago

alltoall(Execspace, ... were missing fences

cedricchevalier19 commented 5 months ago

Currently it is a bit weird, as we don't even require that the view's memory space is accessible from the execution space.

It's a very good point.

Another thing is that the we should probably change most (all?) fences to also have the label.

I have copied one with labels, but we should update all the others. Thanks for noticing.