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

Reply via email to