[Openmp-commits] [PATCH] D56274: [OPENMP][NVPTX]Fix incompatibility of __syncthreads with LLVM, NFC.

George Rokos via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Jan 3 08:38:18 PST 2019


grokos accepted this revision.
grokos added a comment.
This revision is now accepted and ready to land.

I'll accept the patch for the sake of consistency and correctness of execution. Just one question:

> which cost is too high

So should we expect a performance penalty until function copy is fixed in LLVM and we can revert back to `__syncthreads()`?


Repository:
  rOMP OpenMP

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56274/new/

https://reviews.llvm.org/D56274





More information about the Openmp-commits mailing list