Author: Pierre van Houtryve
Date: 2026-03-12T13:58:10+01:00
New Revision: 93b47200f5ea4c3600ad725bb59a77a0b53da45b

URL: 
https://github.com/llvm/llvm-project/commit/93b47200f5ea4c3600ad725bb59a77a0b53da45b
DIFF: 
https://github.com/llvm/llvm-project/commit/93b47200f5ea4c3600ad725bb59a77a0b53da45b.diff

LOG: [AMDGPU] Address post-commit review of #177343 (#186064)

- OpenCL has no cluster scope so "cluster-one-as" does not exist and
cannot be emitted.

Added: 
    

Modified: 
    clang/lib/CodeGen/Targets/AMDGPU.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/CodeGen/Targets/AMDGPU.cpp 
b/clang/lib/CodeGen/Targets/AMDGPU.cpp
index 4ac7f42289d6d..6c7fa2f91aa96 100644
--- a/clang/lib/CodeGen/Targets/AMDGPU.cpp
+++ b/clang/lib/CodeGen/Targets/AMDGPU.cpp
@@ -511,7 +511,8 @@ StringRef AMDGPUTargetCodeGenInfo::getLLVMSyncScopeStr(
     return IsOneAs ? "wavefront-one-as" : "wavefront";
   case SyncScope::HIPCluster:
   case SyncScope::ClusterScope:
-    return IsOneAs ? "cluster-one-as" : "cluster";
+    assert(!IsOneAs && "OpenCL does not have cluster scope");
+    return "cluster";
   case SyncScope::HIPWorkgroup:
   case SyncScope::OpenCLWorkGroup:
   case SyncScope::WorkgroupScope:


        
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to