[PATCH] D93778: [test-suite] Support Fortran tests in CMake infrastructure.

[Nichols A. Romero via Phabricator via llvm-commits]

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