Skip to content
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.

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

[cmake] Disable FatLTO in clang build for Fuchsia #85677

Conversation

ilovepi
Copy link
Contributor

@ilovepi ilovepi commented Mar 18, 2024

We're seeing an issue on Macs, which shouldn't be using this config, so
we will temporarily disable this while we investigate.

Created using spr 1.3.4
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Mar 18, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Mar 18, 2024

@llvm/pr-subscribers-clang

Author: Paul Kirth (ilovepi)

Changes

We're seeing an issue on Macs, which shouldn't be using this config, so
we will temporarily disable this while we investigate.


Full diff: https://github.com/llvm/llvm-project/pull/85677.diff

1 Files Affected:

  • (modified) clang/cmake/caches/Fuchsia-stage2.cmake (+1-1)
diff --git a/clang/cmake/caches/Fuchsia-stage2.cmake b/clang/cmake/caches/Fuchsia-stage2.cmake
index d5546e20873b3c..bc1a05345ce237 100644
--- a/clang/cmake/caches/Fuchsia-stage2.cmake
+++ b/clang/cmake/caches/Fuchsia-stage2.cmake
@@ -11,7 +11,7 @@ set(LLVM_ENABLE_RUNTIMES "compiler-rt;libcxx;libcxxabi;libunwind" CACHE STRING "
 
 set(LLVM_ENABLE_BACKTRACES OFF CACHE BOOL "")
 set(LLVM_ENABLE_DIA_SDK OFF CACHE BOOL "")
-set(LLVM_ENABLE_FATLTO ON CACHE BOOL "")
+set(LLVM_ENABLE_FATLTO OFF CACHE BOOL "")
 set(LLVM_ENABLE_HTTPLIB ON CACHE BOOL "")
 set(LLVM_ENABLE_LIBCXX ON CACHE BOOL "")
 set(LLVM_ENABLE_LIBEDIT OFF CACHE BOOL "")

@ilovepi ilovepi merged commit 457f762 into main Mar 18, 2024
5 of 6 checks passed
@ilovepi ilovepi deleted the users/ilovepi/spr/cmake-disable-fatlto-in-clang-build-for-fuchsia branch March 18, 2024 18:29
chencha3 pushed a commit to chencha3/llvm-project that referenced this pull request Mar 23, 2024
We're seeing an issue on Macs, which shouldn't be using this config, so
we will temporarily disable this while we investigate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants