[Openmp-commits] [PATCH] D48480: [OPENMP, NVPTX] Fixes for NVPTX RTL

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added inline comments.


================
Comment at: libomptarget/deviceRTLs/nvptx/src/sync.cu:36-38
 // Emit a simple barrier call in SPMD mode.  Assumes the caller is in an L0
 // parallel region and that all worker threads participate.
 EXTERN void __kmpc_barrier_simple_spmd(kmp_Indent *loc_ref, int32_t tid) {
----------------
Are you moving these functions for a reason? I think they are defined as extern, so there's no need to have them before calling them.

(Moving code makes reading patches very difficult, I think the only change in this file is in `__kmpc_barrier`, right?)


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D48480