[PATCH] D59494: AMDGPU: Add support for cross address space synchronization scopes

Stanislav Mekhanoshin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 18 10:21:48 PDT 2019


rampitec added inline comments.


================
Comment at: lib/CodeGen/TargetInfo.cpp:7973
+  if (Ordering != llvm::AtomicOrdering::SequentiallyConsistent) {
+    if (Scope != SyncScope::OpenCLAllSVMDevices)
+      Name = Twine(Twine(Name) + Twine("-")).str();
----------------
if (!Name.empty())


================
Comment at: lib/CodeGen/TargetInfo.cpp:7976
+
+    Name = Twine(Twine(Name) + Twine("one-as")).str();
+  }
----------------
I think subgroup is in the single address space even if sequentially consistent.


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

https://reviews.llvm.org/D59494





More information about the cfe-commits mailing list