[Openmp-commits] [PATCH] D114270: [openmp][amdgpu][nfc] Simplify implicit args handling

[Matt Arsenault via Phabricator via Openmp-commits]

arsenm added inline comments.


================
Comment at: openmp/libomptarget/plugins/amdgpu/impl/internal.h:45
 } impl_implicit_args_t;
+static_assert(sizeof(impl_implicit_args_t) == 56, "");
 
----------------
brooks wrote:
> This breaks builds on 32-bit platforms (e.g., i386). If they aren't supported that's fine, but then cmake adjustments are required to prevent it from trying to build.
Just use explicit uint64_t, the named types randomly changing sizes is one of the dumbest things about C


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114270