[Openmp-commits] [PATCH] D68367: Initialise workFn to zero explicitly

Alexey Bataev via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Oct 3 09:15:43 PDT 2019


ABataev added a comment.

In D68367#1693116 <https://reviews.llvm.org/D68367#1693116>, @JonChesterfield wrote:

> I think it needs to be initialized to zero to handle the case where there is no work, though I'm not the original author of the patch.
>
> Iirc cuda/nvcc will implicitly zero initialize this and hip/clang does not.


cuda does not initialize it to zero. If there is no work, __kmpc_kernel_deinit will set it to 0 and terminate the workers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68367





More information about the Openmp-commits mailing list