[Openmp-commits] [PATCH] D74145: [OpenMP][Offloading] Added support for multiple streams so that multiple kernels can be executed concurrently

Sat Feb 8 18:01:55 PST 2020

jdoerfert added a comment.

In D74145#1865927 <https://reviews.llvm.org/D74145#1865927>, @ye-luo wrote:

> In D74145#1865837 <https://reviews.llvm.org/D74145#1865837>, @jdoerfert wrote:
>
> > We will probably need "version 2" functions soon which take additional information, e.g., the stream to be used. I would suggest to test this as is and merge it before we go there. It should already allow overlap between threads that offload. The "version 2" will only shrink the overhead per thread. That said, we are working on the `nowait` support so there will be other changes soon anyway.
> >
> > @ye-luo Do you have a way to test this or do we need to fix the linker issue first?
>
>
> My standalone code can be used to verify multi-stream concurrent execution and whether transfer and execution use the same stream by profiling with nvprof.

What standalone code? Can you run it with this patch? The transfer will use a different stream for now but it should be OK for now. "version 2" will do the same stream.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74145/new/

https://reviews.llvm.org/D74145