[PATCH] D93778: [test-suite] Support Fortran tests in CMake infrastructure.
Nichols A. Romero via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 6 10:39:48 PST 2021
naromero77 added inline comments.
================
Comment at: CMakeLists.txt:7-11
+if (NOT TEST_SUITE_FORTRAN)
+ project(test-suite C CXX)
+else()
+ project(test-suite C CXX Fortran)
+endif()
----------------
naromero77 wrote:
> Meinersbur wrote:
> > Did you consider using `enable_language`? It supports an `OPTIONAL` flag for autodetect the presence of a Fortran compiler. (Not suggesting to use it, it might break buildbots that happen to have gfortran installed, and you will probably not want to run tests for a well-tested distribution provided compiler).
> >
> Yes, I looked into that, but the `OPTIONAL` flag does not work at the moment according to the CMake docs.
>
> https://cmake.org/cmake/help/latest/command/enable_language.html
>
>
I will make this change as well.
Repository:
rT test-suite
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93778/new/
https://reviews.llvm.org/D93778
More information about the llvm-commits
mailing list