[Openmp-commits] [PATCH] D40083: [CMake] Detect information about test compiler

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added a comment.

I hope there is no such restriction in this patch: You should still be able to set `OPENMP_TEST_C_COMPILER` and `OPENMP_TEST_CXX_COMPILER` as before. To get their versions I implemented a nested CMake invocation where `-DCMAKE_C_COMPILER=${OPENMP_TEST_C_COMPILER} -DCMAKE_CXX_COMPILER=${OPENMP_TEST_CXX_COMPILER}` so that we don't have to maintain a version parser for all supported compilers.

The only restriction I added is a check that C and C++ compiler have the same ID - which was assumed anyway and could lead to nasty problems if the user passes in something different.


https://reviews.llvm.org/D40083