[Openmp-commits] [openmp] [openmp][testing] disable check targets if not building with clang (PR #71866)

Maksim Levental via Openmp-commits openmp-commits at lists.llvm.org
Thu Nov 9 17:54:09 PST 2023


makslevental wrote:

> This doesn't look right. Basically, `libomp` can be built with any compatible compiler, while `libomptarget` requires in-tree clang. We can test `libomp` w/o need of clang but indeed we need `clang` to test `libomptarget`. This changes can affect both.

Can you suggest an alternative patch then? Isn't it clearly a bug that I absolutely can't build `libomp` without building in-tree clang even though you concede it is possible and fine?

> This changes can affect both.

I also don't see how this affectst actually anything at all since people wanting to build/test `libomptarget` (e.g., the builder bots) will indeed enable both `clang` and `openmp` projects?

https://github.com/llvm/llvm-project/pull/71866


More information about the Openmp-commits mailing list