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 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
