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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
