saiislam marked an inline comment as done.
saiislam added inline comments.
================
Comment at: openmp/libomptarget/plugins-nextgen/amdgpu/utils/UtilitiesRTL.h:49
-static_assert(sizeof(AMDGPUImplicitArgsTy) == 56,
- "Unexpected size of implicit arguments");
+enum IMPLICITARGS : uint32_t {
+ COV4_SIZE = 56,
----------------
jhuber6 wrote:
> We should probably be using `sizeof` now that it's back to being a struct and
> keep the old struct definition.
AMDGPU plugin doesn't use any implicitarg for COV4, but it does so for COV5.
So, we are not keeping two separate structures for implicitargs of COV4 and
COV5.
If we use sizeof then it will always return 256 corresponding to COV5 (even for
cov4, which should be 56). That's why we need this function.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139730/new/
https://reviews.llvm.org/D139730
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
