[Openmp-commits] [llvm] [openmp] [flang-rt][openmp] Add file-level dependencies for builtin mod files (PR #204260)

[Nikita Popov via Openmp-commits]

================
@@ -291,18 +291,43 @@ if (FLANG_RT_FORTRAN_MODULES)
   # to disable this behavior, unfortunately it does not work with Ninja
   # (https://gitlab.kitware.com/cmake/cmake/-/issues/26803)
   # As a workaround, we build those intrinsic modules first such that the main
-  # runtime can depend on it.
-  add_flangrt_library(flang_rt.mod.intrinsics OBJECT
-    ${intrinsics_sources}
----------------
nikic wrote:

This previously used intrinsics_sources which included __fortran_builtins.f90 and __cuda_builtins.f90, and conditionally on powerpc also __ppc_types.f90.

It looks like the new implementation has handling for the first two, but not the last one.

The intrinsics_sources variable is now not used at all.

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