[all-commits] [llvm/llvm-project] 3fb5c3: [Clang][OpenCL][AMDGPU] Use `byref` for OpenCL ker...

Shilei Tian via All-commits all-commits at lists.llvm.org
Tue Apr 8 10:40:44 PDT 2025


  Branch: refs/heads/users/shiltian/clang-use-byref-for-opencl-kernels
  Home:   https://github.com/llvm/llvm-project
  Commit: 3fb5c3926514abc87b8c0208df1e35e1d1079752
      https://github.com/llvm/llvm-project/commit/3fb5c3926514abc87b8c0208df1e35e1d1079752
  Author: Shilei Tian <i at tianshilei.me>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M clang/lib/CodeGen/Targets/AMDGPU.cpp
    M clang/test/CodeGenOpenCL/addr-space-struct-arg.cl
    M clang/test/CodeGenOpenCL/amdgpu-abi-struct-arg-byref.cl
    M clang/test/CodeGenOpenCL/amdgpu-abi-struct-coerce.cl
    M clang/test/CodeGenOpenCL/opencl-kernel-call.cl

  Log Message:
  -----------
  [Clang][OpenCL][AMDGPU] Use `byref` for OpenCL kernel arguments

Due to a previous workaround allowing kernels to be called from other functions,
Clang currently doesn't use the `byref` attribute for aggregate kernel
arguments. The issue was recently resolved in
https://github.com/llvm/llvm-project/pull/115821. With that fix, we can now
enable the use of `byref` consistently across all languages.

Co-authored-by: Matt Arsenault <Matthew.Arsenault at amd.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list