[Openmp-commits] [PATCH] D154172: [OpenMP] Added memory scope to atomic::inc API and used the device scope in reduction.

Dhruva Chakrabarti via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jun 30 11:26:59 PDT 2023


dhruvachak updated this revision to Diff 536334.
dhruvachak added a comment.

Addressed feedback. Removed the fence change, used the suggested macro, and used the enum scope for clarity.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154172

Files:
  openmp/libomptarget/DeviceRTL/include/Synchronization.h
  openmp/libomptarget/DeviceRTL/src/Reduction.cpp
  openmp/libomptarget/DeviceRTL/src/Synchronization.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D154172.536334.patch
Type: text/x-patch
Size: 5471 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20230630/f5358cc8/attachment.bin>


More information about the Openmp-commits mailing list