mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-22 04:25:13 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Lagom: Link LibMain statically

Browse source 
This matches how LibMain is used within Serenity. This commit makes it
possible to build Lagom with LTO. Previously, `serenity_main` functions
would be dead-stripped away, as the linker could prove that nothing from
the executable ever called them.
This commit is contained in:
Daniel Bertalan 2022-10-01 16:08:26 +02:00 • committed by Andreas Kling
parent add7dd66f6
commit 41a8f3f40f
Notes: sideshowbarker 2024-07-17 07:06:47 +09:00
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
Meta/Lagom/CMakeLists.txt
View file

@ -171,9 +171,13 @@ install(
)
function(lagom_lib library fs_name)
cmake_parse_arguments(LAGOM_LIBRARY "" "" "SOURCES;LIBS" ${ARGN})
cmake_parse_arguments(LAGOM_LIBRARY "STATIC" "" "SOURCES;LIBS" ${ARGN})
set(target_name "Lib${library}")
add_library(${target_name} ${LAGOM_LIBRARY_SOURCES})
if (LAGOM_LIBRARY_STATIC)
add_library(${target_name} STATIC ${LAGOM_LIBRARY_SOURCES})
else()
add_library(${target_name} ${LAGOM_LIBRARY_SOURCES})
endif()
# Don't make alias when we're going to import a previous build for Tools
# FIXME: Is there a better way to write this?
@ -283,11 +287,8 @@ endif()
file(GLOB LIBMAIN_SOURCES CONFIGURE_DEPENDS "../../Userland/Libraries/LibMain/*.cpp")
lagom_lib(Main main
SOURCES ${LIBMAIN_SOURCES}
STATIC
)
# The macOS linker is not happy about LibMain's main() calling an undefined symbol (serenity_main()).
if (APPLE)
target_link_options(LibMain PRIVATE -undefined dynamic_lookup)
endif()
# LibTimeZone
# This is needed even if Lagom is not enabled because it is depended upon by code generators.