Page MenuHomePhabricator

[builtins] Build both static and shared versions of the builtins library.
AbandonedPublic

Authored by vkalintiris on Sep 24 2015, 9:40 AM.

Diff Detail

Event Timeline

vkalintiris retitled this revision from to [builtins] Build both static and shared versions of the builtins library..
vkalintiris updated this object.
vkalintiris added a subscriber: llvm-commits.
vkalintiris added inline comments.Sep 24 2015, 9:44 AM
lib/builtins/CMakeLists.txt
343

Why do we use the C99 standard explicitly here? I would like to change that and use whatever is the compiler's default or make it a user defined option. At least for MIPS, this would allow us to include atomic.c in builtins, when using C11.

samsonov edited edge metadata.Oct 5 2015, 5:15 PM

Do you have a proper support (i.e. Clang driver logic, and compiler flag) for using clang_rt.builtins shared runtime? Or for now you just somehow link with shared compiler-rt runtime manually?

rengolin edited edge metadata.Oct 6 2015, 6:52 AM

What's the motivation behind this? I'm not saying I'm against it, just want to understand the scope and objectives, and make sure we can support (and test) all configurations.

What's the motivation behind this? I'm not saying I'm against it, just want to understand the scope and objectives, and make sure we can support (and test) all configurations.

I just thought that it would be an alternative for libgcc_s for toolchains that don't depend on a GCC installation. Currently, I'm not aware of any Clang toolchain that can use this shared library. I'd like to add support for using the shared build of the builtins library by the mips-mti-linux (D13340) toolchain in the next few weeks. Would it be better to wait until then?

Yes, let's postpone this patch until Clang is able to properly use this shared library, and we have means to test it.

vkalintiris abandoned this revision.Jun 5 2016, 10:39 AM