mirror of
https://github.com/electron/electron.git
synced 2026-04-10 03:01:51 -04:00
699445f5da
* chore: bump chromium in DEPS to 147.0.7705.0 * chore: update patches (trivial only) * 7607719: Refactor predictors GN targets and clean up LCPP unittest params Ref: https://chromium-review.googlesource.com/c/chromium/src/+/7607719 Co-Authored-By: Claude (claude-3-5-sonnet) --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: John Kleinschmidt <kleinschmidtorama@gmail.com>
26 lines
1.2 KiB
Diff
26 lines
1.2 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: VerteDinde <vertedinde@electronjs.org>
|
|
Date: Tue, 12 Nov 2024 21:16:15 -0800
|
|
Subject: build: disable thin_lto on mac
|
|
|
|
This patch disables thin lto on mac only. After Node 22 was merged into
|
|
main, release builds began failing due to missing symbols, which seem to
|
|
be being stripped out by thin lto.
|
|
|
|
This patch can (and should) be removed when we can prevent those symbols
|
|
from being stripped in the release build.
|
|
|
|
diff --git a/build/config/compiler/compiler.gni b/build/config/compiler/compiler.gni
|
|
index 7e56497d4809c1946e636f3210c67575f8ae84bd..7ad7d3594890bcc9774f22c0cb3866a7e2b855fc 100644
|
|
--- a/build/config/compiler/compiler.gni
|
|
+++ b/build/config/compiler/compiler.gni
|
|
@@ -100,7 +100,7 @@ declare_args() {
|
|
# Chrome's clang. crbug.com/1033839
|
|
use_thin_lto =
|
|
is_cfi || (is_clang && is_official_build && chrome_pgo_phase != 1 &&
|
|
- (is_linux || is_win || is_mac || (is_ios && use_lld) ||
|
|
+ (is_linux || is_win || (is_ios && use_lld) ||
|
|
(is_android && target_os != "chromeos") ||
|
|
(is_chromeos && is_chromeos_device)))
|
|
|