tra added inline comments.
================ Comment at: clang/lib/CodeGen/CGCall.cpp:1172 if (isa<llvm::PointerType>(Ty)) - return CGF.Builder.CreateBitCast(Val, Ty, "coerce.val"); + return CGF.Builder.CreatePointerCast(Val, Ty, "coerce.val"); ---------------- What's the purpose of this change? ================ Comment at: clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu:15-16 +// CHECK: define amdgpu_kernel void @_Z7kernel3PU3AS2iPU3AS1i(i32 addrspace(2)* %x, i32 addrspace(1)* %y) +__global__ void kernel3(__attribute__((address_space(2))) int *x, + __attribute__((address_space(1))) int *y) { + y[0] = x[0]; ---------------- Interesting. Clang used to crash on explicit address space attribute in the past. I'm glad it works now. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69818/new/ https://reviews.llvm.org/D69818 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits