[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;
> 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
CHANGES SINCE LAST ACTION
More information about the Openmp-commits