[Openmp-commits] [PATCH] D51875: [OPENMP][NVPTX] Add support for lastprivates/reductions handling in SPMD constructs with lightweight runtime.
Alexey Bataev via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Mon Sep 10 13:19:12 PDT 2018
ABataev added a comment.
In https://reviews.llvm.org/D51875#1229502, @Hahnfeld wrote:
> In https://reviews.llvm.org/D51875#1229496, @ABataev wrote:
> > In https://reviews.llvm.org/D51875#1229491, @Hahnfeld wrote:
> > > I really, really dislike adding even more global buffers. `4096 * 32 * 56` are another 7MiB that are not usable for applications. What's wrong with using the existing ones?
> > >
> > > Can you upload the CodeGen patch for reductions somewhere? I thought we need a global scratchpad buffer that is adressable for all teams?
> > I really, really dislike an implementation in ibm-devel, the scratchpad solution will never be added to the trunk. The existing ones cannot be reused, as they are allocated only if the full runtime is used.
> What's the overhead of initializing it? The whole `libomptarget-nvptx` is already a pretty much mess, see my thread on openmp-dev.
It is not the runtime issue, it is the problem with the compiler itself. It breaks compatibility with the other outlined regions and, thus, it cannot be committed to trunk. I'd like to have this at least as a temporirily solution to support lastprivates/reductions in SPMD mode with lightweight runtime. We can reduce the size of the preallocated buffers, if you wish.
More information about the Openmp-commits