[Openmp-commits] [openmp] [OpenMP][omptest] Improve CMake and address review comments (PR #159416)

[Michael Kruse via Openmp-commits]

Meinersbur wrote:

Some thoughts: To work around conflicting ABI between the main build of LLVM_ENABLE_PROJECTS and nested LLVM_ENABLE_RUNTIMES projects that occur even with `runtimes-bins` default builds (e.g. different libc++/libstdc++ that broke [flang-aarch64-libcxx](https://lab.llvm.org/buildbot/#/builders/89)), I think we should generally not use llvm_gtest in runtimes builds, but like here use `GTEST_NO_LLVM_SUPPORT` to not have that LLVMSupport dependency at all. I would add such a runtimes-GTest build into https://github.com/llvm/llvm-project/blob/main/runtimes/CMakeLists.txt for any runtime to use. However, openmp still support standalone and LLVM_ENABLE_PROJECTS builds which does not use runtimes/CMakeLists.txt.

@DavidSpickett What do you think?

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