[clang] [compiler-rt] [libc] [libcxx] [llvm] [AMDGPU] Fix code object version not being set to 'none' (PR #135036)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 9 10:09:50 PDT 2025


================
@@ -62,7 +62,7 @@ Value *EmitAMDGPUWorkGroupSize(CodeGenFunction &CGF, unsigned Index) {
 
   auto Cov = CGF.getTarget().getTargetOpts().CodeObjectVersion;
 
-  if (Cov == CodeObjectVersionKind::COV_None) {
+  if (Cov == CodeObjectVersionKind::COV_None && !CGF.getLangOpts().OpenMP) {
----------------
jhuber6 wrote:

It's added in https://github.com/llvm/llvm-project/blob/1aed6ad906f54d6d3fd94b7f46aaeb4cc7e516ec/clang/lib/CodeGen/CodeGenModule.cpp#L924. Could I maybe just omit this if the user is compiling with something like `-nogpulib`? Otherwise, we just need another flag for it because `None` is pulling double duty here by doing the reduction.

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


More information about the llvm-commits mailing list