[Openmp-commits] [PATCH] D112153: [OpenMP] Introduce aligned synchronization into the new device RT

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Oct 21 00:51:12 PDT 2021


JonChesterfield added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/src/Parallelism.cpp:97
   if (mapping::isSPMDMode()) {
-    synchronize::threads();
+    // Avoid the race between the read of the `icv::Level` above and the write
+    // below by synchronizing all threads here.
----------------
Thank you for the comments here


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112153



More information about the Openmp-commits mailing list