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

[Sameer Sahasrabuddhe via llvm-branch-commits]

================
@@ -76,11 +77,21 @@ static TargetExtType *getTargetExtType(const GlobalVariable &GV) {
 }
 
 TargetExtType *isNamedBarrier(const GlobalVariable &GV) {
----------------
ssahasra wrote:

The function name suggests it should return a boolean predicate. Should be renamed to `getBarrierType`. Now I am wondering if `AMDGPUNamedBarrier` should be a subclass of `GlobalVariable` that supports `isa`, `cast` etc.

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