[PATCH] D55067: [HIP] Fix offset of kernel argument for AMDGPU target

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 29 11:52:09 PST 2018


rjmccall added a comment.

This seems backwards.  Clang knows what the actual ABI alignment of the C type is, and it doesn't have to match the alignment of the IR type that IRGen produces.  It's the actual C ABI alignment that's supposed to affect the calling convention, so there needs to be some way to specify the C ABI alignment on the parameter in IR.  That may mean using `byval`, which can be given an explicit alignment.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55067/new/

https://reviews.llvm.org/D55067





More information about the cfe-commits mailing list