[llvm-branch-commits] [clang] [flang] [lld] [llvm] [Flang][DRAFT] LLVM_ENABLE_RUNTIMES=FortranRuntime (PR #110217)

Michael Kruse via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Oct 1 08:39:10 PDT 2024


Meinersbur wrote:

I reverted the change that moved `flang/module/*.f90` to `FortranRuntime/module/*.f90`. A lot of Flang tests depend on the `*.mod` files to be available. Moving the modules to FortranRuntime would require moving those tests to FortranRuntime as well to avoid a circular dependency. Except the PPC modules, the mod files are not target-specific and hence would not profit from compiling multiple targets from a bootstrap build anyway. 

However, they are products of the Flang compiler and therefore IMHO logically fits better to the runtime. Also, it would have been nice to make use of CMake's dependency resolution than hardcoding Into the CMakeLists.txt. In any case, such a change can be applied later.

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


More information about the llvm-branch-commits mailing list