[Openmp-commits] [PATCH] D77005: [OpenMP] Optimized stream selection by scheduling data mapping for the same target region into a same stream

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Mar 31 13:09:56 PDT 2020

jdoerfert added inline comments.

Comment at: openmp/libomptarget/include/omptarget.h:118
+  void *Identifier;
tianshilei1992 wrote:
> jdoerfert wrote:
> > tianshilei1992 wrote:
> > > jdoerfert wrote:
> > > > I don't like `Identifier` as name. Maybe execution queue? Or the name we used for streams before? Also, add a comment what it is and why it is a void *
> > > The basic idea is, this structure is device independent. For CUDA device, we use it as `CUstream` so yes, we can call it stream. However, we don't know how it can be used in other platforms, and that's why I use a "weird" word `Identifier` and a `void *` here.
> > We need a better word and documentation of the member. Also initialize it here
> Sure. Do you have any suggestion for the name?

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list