[PATCH] D107971: [openmp] Annotate tmp variables with omp_thread_mem_alloc

[Johannes Doerfert via Phabricator via cfe-commits]

jdoerfert added a comment.

In D107971#2941739 <https://reviews.llvm.org/D107971#2941739>, @JonChesterfield wrote:

> Failed a CI job that builds an openmp test in an environment without omp.h, will revert.
>
> Thoughts on fixing? Putting the omp allocator definition in this header is likely to collide with a real omp.h. Fairly clean fix is to move the definitions into the deviceRTL. Tempting fix is to add a minimal omp.h to the test dir
>
> example failure:
>
>   In file included from /home/tcwg-buildslave/worker/clang-aarch64-quick/llvm/clang/test/Headers/amdgcn_openmp_device_math.c:9:
>   In file included from /home/tcwg-buildslave/worker/clang-aarch64-quick/llvm/clang/test/Headers/../../lib/Headers/openmp_wrappers/math.h:55:
>   /home/tcwg-buildslave/worker/clang-aarch64-quick/stage1/lib/clang/14.0.0/include/__clang_hip_math.h:23:10: fatal error: 'omp.h' file not found
>   #include <omp.h>

yes add `omp.h`, in `clang/test/Headers/Inputs/include`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107971