[llvm-branch-commits] [llvm] [openmp] [OpenMP][Offload] Add offload runtime support for dyn_groupprivate clause (PR #152831)

[Michael Klemm via llvm-branch-commits]

================
@@ -107,7 +107,7 @@ enum TargetAllocTy : int32_t {
 
 inline KernelArgsTy CTorDTorKernelArgs = {1,       0,       nullptr,   nullptr,
             nullptr, nullptr, nullptr,   nullptr,
-            0,      {0,0,0},       {1, 0, 0}, {1, 0, 0}, 0};
+            0,      {0,0,0,0},       {1, 0, 0}, {1, 0, 0}, 0};
----------------
mjklemm wrote:

I think it would make sense to add comments to what these fields mean.  Line 
110 is full of magic constants.

https://github.com/llvm/llvm-project/pull/152831
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits