Skip to content

[Fuchsia][cmake] Allow using FatLTO when building runtimes #112277

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Dec 9, 2024
Merged

[Fuchsia][cmake] Allow using FatLTO when building runtimes #112277

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1dafa52
[𝘀𝗽𝗿] initial version
ilovepi Oct 14, 2024
350d75e
Rebase on main
ilovepi Oct 14, 2024
38851d2
Remove newline from diff
ilovepi Oct 14, 2024
3990524
Rebase on main
ilovepi Oct 14, 2024
535f2f2
Avoid unecessary changes to libc++ cmake
ilovepi Oct 16, 2024
2ad6038
[𝘀𝗽𝗿] changes introduced through rebase
ilovepi Nov 12, 2024
49ce824
rebase and enable FatLTO for Fuchsia targets
ilovepi Nov 12, 2024
9511215
[𝘀𝗽𝗿] changes introduced through rebase
ilovepi Nov 14, 2024
1b3c34f
Rebase
ilovepi Nov 14, 2024
86c376e
[𝘀𝗽𝗿] changes introduced through rebase
ilovepi Nov 14, 2024
13973d0
Move --as-needed into the correct patch
ilovepi Nov 14, 2024
4b7b3f9
[𝘀𝗽𝗿] changes introduced through rebase
ilovepi Nov 18, 2024
b5551cc
update base revision
ilovepi Nov 18, 2024
e0d6cbf
[𝘀𝗽𝗿] changes introduced through rebase
ilovepi Dec 9, 2024
52c567e
Rebase on main
ilovepi Dec 9, 2024
8118e04
Rebase
ilovepi Dec 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions clang/cmake/caches/Fuchsia-stage2.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,10 @@ foreach(target aarch64-unknown-linux-gnu;armv7-unknown-linux-gnueabihf;i386-unkn
set(RUNTIMES_${target}_LLVM_TOOLS_DIR "${CMAKE_BINARY_DIR}/bin" CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_RUNTIMES "compiler-rt;libcxx;libcxxabi;libunwind" CACHE STRING "")

# Enable FatLTO for Linux and baremetal runtimes
set(RUNTIMES_${target}_LLVM_ENABLE_LTO ON CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_FATLTO ON CACHE BOOL "")

# Use .build-id link.
list(APPEND RUNTIME_BUILD_ID_LINK "${target}")
endif()
Expand Down Expand Up @@ -272,6 +276,10 @@ if(FUCHSIA_SDK)
set(RUNTIMES_${target}+asan+noexcept_LIBCXXABI_ENABLE_EXCEPTIONS OFF CACHE BOOL "")
set(RUNTIMES_${target}+asan+noexcept_LIBCXX_ENABLE_EXCEPTIONS OFF CACHE BOOL "")

# Enable FatLTO for Fuchsia runtimes
set(RUNTIMES_${target}_LLVM_ENABLE_LTO ON CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_FATLTO ON CACHE BOOL "")

# Use .build-id link.
list(APPEND RUNTIME_BUILD_ID_LINK "${target}")
endforeach()
Expand Down Expand Up @@ -363,6 +371,10 @@ foreach(target armv6m-none-eabi;armv7m-none-eabi;armv8m.main-none-eabi;armv8.1m.
set(RUNTIMES_${target}_LLVM_INCLUDE_TESTS OFF CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_ASSERTIONS OFF CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_RUNTIMES "libc;libcxx" CACHE STRING "")

# Enable FatLTO for baremetal runtimes
set(RUNTIMES_${target}_LLVM_ENABLE_LTO ON CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_FATLTO ON CACHE BOOL "")
endforeach()

foreach(target riscv32-unknown-elf)
Expand Down Expand Up @@ -414,6 +426,10 @@ foreach(target riscv32-unknown-elf)
set(RUNTIMES_${target}_LLVM_INCLUDE_TESTS OFF CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_ASSERTIONS OFF CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_RUNTIMES "libc;libcxx" CACHE STRING "")

# Enable FatLTO for baremetal runtimes
set(RUNTIMES_${target}_LLVM_ENABLE_LTO ON CACHE BOOL "")
set(RUNTIMES_${target}_LLVM_ENABLE_FATLTO ON CACHE BOOL "")
endforeach()

set(LLVM_BUILTIN_TARGETS "${BUILTIN_TARGETS}" CACHE STRING "")
Expand Down
2 changes: 1 addition & 1 deletion llvm/cmake/modules/HandleLLVMOptions.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1285,7 +1285,7 @@ elseif(LLVM_ENABLE_LTO)
endif()
endif()

if(LLVM_ENABLE_FATLTO AND UNIX AND NOT APPLE)
if(LLVM_ENABLE_FATLTO AND ((UNIX AND NOT APPLE) OR FUCHSIA))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why that diff is required since you're setting it properly in your cache files.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because Fuchsia Targets aren't UNIX. Without this change -ffat-lto-objects won't be passed to clang through CMake. The additional logic is basically to restrict the flag to ELF targets, and it missed Fuchsia when we added it, since we weren't building our runtimes with FatLTO initially, just the toolchain + tests.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I think I initially read if (LLVM_ENABLE_FATLTO OR ((UNIX AND NOT APPLE) OR FUCHSIA)).

append("-ffat-lto-objects" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
if(NOT LINKER_IS_LLD_LINK)
append("-ffat-lto-objects" CMAKE_EXE_LINKER_FLAGS CMAKE_SHARED_LINKER_FLAGS CMAKE_MODULE_LINKER_FLAGS)
Expand Down
Loading