[clang] [Clang][AMDGPU] Avoid Using `byval` for `ndrange_t` when emitting `__enqueue_kernel_basic` (PR #116435)

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 15 12:16:29 PST 2024


================
@@ -5985,10 +5985,14 @@ RValue CodeGenFunction::EmitBuiltinExpr(const GlobalDecl GD, unsigned BuiltinID,
       llvm::Value *Block =
           Builder.CreatePointerCast(Info.BlockArg, GenericVoidPtrTy);
 
-      AttrBuilder B(Builder.getContext());
-      B.addByValAttr(NDRangeL.getAddress().getElementType());
-      llvm::AttributeList ByValAttrSet =
-          llvm::AttributeList::get(CGM.getModule().getContext(), 3U, B);
+      llvm::AttributeList ByValAttrSet;
+      // AMDGPU doesn't use byval for struct argument.
----------------
arsenm wrote:

Should query the ABInfo for what to do for the signature 

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


More information about the cfe-commits mailing list