[Openmp-commits] [PATCH] D100673: [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault.

[Alexey Bataev via Phabricator via Openmp-commits]

ABataev added a comment.

In D100673#2716929 <https://reviews.llvm.org/D100673#2716929>, @jdoerfert wrote:

> In D100673#2716916 <https://reviews.llvm.org/D100673#2716916>, @ABataev wrote:
>
>> In D100673#2714087 <https://reviews.llvm.org/D100673#2714087>, @jdoerfert wrote:
>>
>>> I think this broke GridMini (https://github.com/meifeng/GridMini/tree/openmp) :(
>>
>> Hmm, this app does not use mappers. Are you sure that this commit is the actual cause of the problem? Also, could you provide detailed debug log for the investigation (with LIBOMPTARGET_DEBUG set)?
>
> I did debug something else and the clang-omp branch failed until I reverted this patch, at least that is what I thought. I will look again as soon as I find the time. For sure it run fine until very recently and we should make sure it continues to do so ;)

Definitely! That's why I'm asking for the debug log for the investigation. It requires CUDA but I don't have an access to the machine with cuda.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100673