Skip to contentSkip to conversation

package/llvm: add dependency on shared libraries

PATCH New Valentin Korenblit 3 participants 5 messages Apr 6, 2018

lore
Share & tools
lore

Valentin Korenblit

author
#0 lore raw

PATCH 1/1 package/llvm: add dependency on shared libraries

llvm generates libLLVM.so, so it won't work if BR2_STATIC_LIBS is set

Signed-off-by Valentin Korenblit <[email protected]>
---
 package/llvm/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/llvm/Config.in +1 -0
index 70f73f4858..d507acbef1 100644
@@ -18,6 +18,7 @@ config BR2_PACKAGE_LLVM
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_INSTALL_LIBSTDCPP
+	depends on !BR2_STATIC_LIBS
 	help
 	  The LLVM Project is a collection of modular and reusable
 	  compiler and toolchain technologies.
Signature · 1 line
2.14.3
#1 lore raw

PATCH 1/1 package/llvm: add dependency on shared libraries

Hi Valentin,

Le 06/04/2018 ? 11:39, Valentin Korenblit a ?crit?:

Quoted 12 lines

llvm generates libLLVM.so, so it won't work if BR2_STATIC_LIBS is set

Signed-off-by Valentin Korenblit <[email protected]>
---
 package/llvm/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/llvm/Config.in +0 -0
index 70f73f4858..d507acbef1 100644

package/llvm/Config.in config BR2_PACKAGE_LLVM · line 21

Quoted 4 lines
package/llvm/Config.in +1 -0
@@ -18,6 +18,7 @@ config BR2_PACKAGE_LLVM
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_INSTALL_LIBSTDCPP
+	depends on !BR2_STATIC_LIBS

This new dependency must be propagated to the comment below and other options selecting llvm:

comment "llvm needs a toolchain w/ threads, C++, dynamic library, gcc >= 4.8, host gcc >= 4.8"

	depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS
	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP || \
                !BR2_STATIC_LIBS \
		!BR2_HOST_GCC_AT_LEAST_4_8 || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
Quoted 4 lines
package/llvm/Config.in +0 -0
 	help
 	  The LLVM Project is a collection of modular and reusable
 	  compiler and toolchain technologies.
#2 lore raw

PATCH 1/1 package/llvm: add dependency on shared libraries

In reply to Valentin Korenblit

Hello,

Quoted 3 lines · Valentin Korenblit

llvm generates libLLVM.so, so it won't work if BR2_STATIC_LIBS is set

Signed-off-by Valentin Korenblit <[email protected]>

Interestingly, I haven't seen any failure of LLVM related to static libraries reported by the autobuilders.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
author
#3 lore raw

PATCH 1/1 package/llvm: add dependency on shared libraries

Hello Thomas,

Quoted 8 lines · Thomas Petazzoni

Hello,

Quoted 3 lines · Valentin Korenblit

llvm generates libLLVM.so, so it won't work if BR2_STATIC_LIBS is set

Signed-off-by Valentin Korenblit <[email protected]>

Interestingly, I haven't seen any failure of LLVM related to static libraries reported by the autobuilders.

I think llvm should compile correctly but mesa3d has --enable-llvm-shared-libs. We can disable that option if BR2_STATIC_ONLY.

Quoted 1 line

Thomas

Valentin

#4 lore raw

PATCH 1/1 package/llvm: add dependency on shared libraries

Hello,

Quoted 6 lines · Valentin Korenblit
Quoted 3 lines
Quoted 1 line
Signed-off-by Valentin Korenblit <[email protected]>

Interestingly, I haven't seen any failure of LLVM related to static libraries reported by the autobuilders.

I think llvm should compile correctly but mesa3d has --enable-llvm-shared-libs. We can disable that option if BR2_STATIC_ONLY.

LLVM doesn't build properly with BR2_STATIC_LIBS. The following defconfig:

BR2_arm=y
BR2_cortex_a8=y
BR2_STATIC_LIBS=y
BR2_TOOLCHAIN_BUILDROOT_LOCALE=y
BR2_TOOLCHAIN_BUILDROOT_CXX=y
BR2_PACKAGE_LLVM=y

fails to build with:

[ 60%] Linking CXX shared module ../../LLVMHello.so
/home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/bin/ld: /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../ ../arm-buildroot-linux-uclibcgnueabihf/lib/libstdc++.a(eh_globals.o)(.text.__cxa_get_globals_fast+0x10): R_ARM_TLS_LE32 relocation not permitted in shared object /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/lib/libstdc++.a(eh_globals.o): In function `__cxa_get_globals_fast':
eh_globals.cc:(.text.__cxa_get_globals_fast+0x10): dangerous relocation: unsupported relocation /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/bin/ld: /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../ ../arm-buildroot-linux-uclibcgnueabihf/lib/libstdc++.a(eh_globals.o)(.text.__cxa_get_globals+0x10): R_ARM_TLS_LE32 relocation not permitted in shared object /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/lib/libstdc++.a(eh_globals.o): In function `__cxa_get_globals':
eh_globals.cc:(.text.__cxa_get_globals+0x10): dangerous relocation: unsupported relocation
/home/thomas/buildroot/buildroot/output/host/arm-buildroot-linux-uclibcgnueabihf/sysroot/usr/lib/libc.a(__uClibc_main.os): In function `__uClibc_fini':
__uClibc_main.c:(.text+0x164): undefined reference to `__fini_array_end'
__uClibc_main.c:(.text+0x168): undefined reference to `__fini_array_start'
__uClibc_main.c:(.text+0x16c): undefined reference to `__fini_array_start'
/home/thomas/buildroot/buildroot/output/host/arm-buildroot-linux-uclibcgnueabihf/sysroot/usr/lib/libc.a(__uClibc_main.os): In function `__uClibc_main':
__uClibc_main.c:(.text+0x580): undefined reference to `__preinit_array_start'
__uClibc_main.c:(.text+0x584): undefined reference to `__preinit_array_start'
__uClibc_main.c:(.text+0x588): undefined reference to `__preinit_array_end'
__uClibc_main.c:(.text+0x58c): undefined reference to `__init_array_start'
__uClibc_main.c:(.text+0x590): undefined reference to `__init_array_end'
__uClibc_main.c:(.text+0x594): undefined reference to `__init_array_start' /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/bin/ld: ../../LLVMHello.so: hidden symbol `__fini_array_end' isn't defined /home/thomas/buildroot/buildroot/output/host/lib/gcc/arm-buildroot-linux-uclibcgnueabihf/6.4.0/../../../../arm-buildroot-linux-uclibcgnueabihf/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status
lib/Transforms/Hello/CMakeFiles/LLVMHello.dir/build.make:94: recipe for target 'lib/LLVMHello.so' failed
make[4]: *** [lib/LLVMHello.so] Error 1

So basically, it's trying to build a shared library.

Could you fix up your patch to propagate the !BR2_STATIC_LIBS dependency to:

1. The Config.in comment in package/llvm/Config.in

2. The mesa3d package that selects BR2_PACKAGE_LLVM

And send an updated version ?

Thanks!

Thomas Petazzoni

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com