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

[Michael Kruse via llvm-branch-commits]

Meinersbur wrote:

> Thanks for your inputs @Meinersbur! [a036068](https://github.com/llvm/llvm-project/commit/a03606822b0eaef8efdeb00c4f2c33e1a029f79f) fixed the compilation issues on linux.
> 
> I have another question - when building the runtimes (separately as you described), it seems that `-DBUILD_SHARED_LIBS=ON` gets ignored also on linux, in the sense that the resulting library is `libflang_rt.a` and not `libflang_rt.so`. You had commented that this is a known limitation on windows, but AFAIU this should still work on linux (as previously)?

IMHO whether building shared or static libs should not depend on a configuration parameter. That adds to the configuration space explosion. It is also not an either-or-choice, you regularly want both. So like for the Windows libraries, Flang-RT should build both at the same time. As written in the RFC, I am going to add support for also building the shared library after this. Complexity comes with whether non-PIC libraries are needed, otherwise we can use OBJECT libraries like `llvm_add_library` does when building shared and static at the same time.


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