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

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Oct 20 15:35:16 PDT 2021


jdoerfert added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/src/Synchronization.cpp:186
+  constexpr int BarrierNo = 9;
+  asm volatile("barrier.sync.aligned %0;" : : "r"(BarrierNo) : "memory");
+}
----------------
tianshilei1992 wrote:
> We don't have to use a named barrier for aligned barrier, do we?
I guess not.


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