search for: d59328

...gt; this so I am open to suggestion on how to split it up. >>> >>> >> >> 1. You definitely need to split it into separate patches for different >> components. > > Done: > OpenMP: https://reviews.llvm.org/D59319 > Clang: https://reviews.llvm.org/D59328 > LLVM: https://reviews.llvm.org/D59331 > >> 2. Even inside of those components this patch must be split into several >> small patches, it is very hard to review so big patches. > > Please take a look at the three patches above. > > The first contains the interf...

...gt;> this so I am open to suggestion on how to split it up. >>> >>> >> >> 1. You definitely need to split it into separate patches for different >> components. > > Done: > OpenMP: https://reviews.llvm.org/D59319 > Clang: https://reviews.llvm.org/D59328 > LLVM: https://reviews.llvm.org/D59331 > >> 2. Even inside of those components this patch must be split into several >> small patches, it is very hard to review so big patches. > > Please take a look at the three patches above. > > The first contains the interface...

------------- Best regards, Alexey Bataev 13.03.2019 15:35, Doerfert, Johannes пишет: > > Hi Alexey, > > > thank you for your quick feedback. > > > > There are tooooooo(!) many changes, I don't who's going to review sooooo big > patch. > > > I can for sure split it in the three components/repositories that are > touched, clang, llvm, and openmp.

Where we are ------------ Currently, when we generate OpenMP target offloading code for GPUs, we use sufficient syntactic criteria to decide between two execution modes: 1) SPMD -- All target threads (in an OpenMP team) run all the code. 2) "Guarded" -- The master thread (of an OpenMP team) runs the user code. If an OpenMP distribute region is encountered,