[Openmp-commits] [PATCH] D114270: [openmp][amdgpu][nfc] Simplify implicit args handling
Matt Arsenault via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Mon Jan 10 10:24:49 PST 2022
arsenm added inline comments.
Comment at: openmp/libomptarget/plugins/amdgpu/impl/internal.h:45
+static_assert(sizeof(impl_implicit_args_t) == 56, "");
> 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
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the Openmp-commits