mirror of
https://github.com/genodelabs/genode.git
synced 2024-12-26 08:51:08 +00:00
c38c2a6455
When rebasing my local branch on top of sculpt-21.10 tag I've noticed two problems. The code in new_delete.cc does not include new header file. This works fine with GCC, but fails with clang because std::align_val_t type is not defined anywhere according to clang. It looks like GCC pulls this header indirectly somehow. The second problem can be seen if one disallows undefined symbols in executables and shared_libraries. This can be seen with both GCC and clang by adding --no-undefined to LD_OPT. With such change in place core fails to link due to: ld.lld: error: undefined symbol: operator delete(void*, std::align_val_t) >>> referenced by thread.h:448 (/home/tworaz/devel/genode/repos/base-hw/src/core/kernel/thread.h:448) >>> thread.o:(Kernel::Core_main_thread::~Core_main_thread()) in archive debug/core-hw-virt_qemu.a >>> referenced by thread.h:448 (/home/tworaz/devel/genode/repos/base-hw/src/core/kernel/thread.h:448) >>> thread.o:(non-virtual thunk to Kernel::Core_main_thread::~Core_main_thread()) in archive debug/core-hw-virt_qemu.a >>> did you mean: operator delete(void*, unsigned long, std::align_val_t) >>> defined in: debug/core-hw-virt_qemu.a(supc++.o) If the code would somehow manage call such undefined symbol it'd crash. Since I generally prefer link time failures to runtime crashes I link all genode binaries with --no-undefined. To fix this problem just add a dummy implementation of missing delete operator. Fixes #4298
18 lines
566 B
Makefile
18 lines
566 B
Makefile
SRC_CC += lx_hybrid.cc new_delete.cc capability_space.cc
|
|
SRC_CC += signal_transmitter.cc signal.cc
|
|
SRC_C += libgcc.c
|
|
|
|
# new_delete.cc uses libsupc++ which means we need to access
|
|
# its include directory.
|
|
STDINC := yes
|
|
|
|
vpath new_delete.cc $(BASE_DIR)/src/lib/cxx
|
|
vpath lx_hybrid.cc $(REP_DIR)/src/lib/lx_hybrid
|
|
vpath libgcc.c $(REP_DIR)/src/lib/lx_hybrid
|
|
|
|
# add parts of the base library that are shared with core
|
|
LIBS += base-linux-common timeout
|
|
|
|
# non-core parts of the base library (except for the startup code)
|
|
include $(REP_DIR)/lib/mk/base-linux.inc
|