[llvm-branch-commits] [clang] [lld] [llvm] [mlir] [RFC][AMDGPU] Add BARRIER address space (PR #209746)

[Pierre van Houtryve via llvm-branch-commits]

================
@@ -165,6 +157,26 @@ unsigned AMDGPUMachineFunctionInfo::allocateLDSGlobal(const DataLayout &DL,
   return Offset;
 }
 
+unsigned
+AMDGPUMachineFunctionInfo::allocateBarrierGlobal(const DataLayout &DL,
+                                                 const GlobalVariable &GV) {
+  assert(AMDGPU::isNamedBarrier(GV));
+  std::optional<unsigned> BarAddr =
+      get32BitAbsoluteAddress(GV, AMDGPUAS::BARRIER);
+  if (!BarAddr)
+    llvm_unreachable("named barrier should have an assigned address");
----------------
Pierre-vh wrote:

Good point. The lack of address is an assert as that should just not happen, but I left the check for null as an error

https://github.com/llvm/llvm-project/pull/209746