From 7815f44b24ddbb82ce9d39ab98ac83b6d0b44b3a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Stan=C4=9Bk?= <jstanek@redhat.com>
Date: Tue, 11 Jul 2023 17:30:34 +0200
Subject: [PATCH] Enable LTO during build

---
 nodejs.spec | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/nodejs.spec b/nodejs.spec
index c1bc216..9926473 100644
--- a/nodejs.spec
+++ b/nodejs.spec
@@ -18,7 +18,8 @@
 
 %bcond_with python3_fixup
 
-# LTO is currently broken on Node.js builds
+# NodeJS uses it's own set of LTO flags, and is generally not compatible
+# with globally enabling them; use the --enable-lto configure flag instead.
 %define _lto_cflags %{nil}
 
 # Heavy-handed approach to avoiding issues with python
@@ -425,6 +426,7 @@ export CFLAGS="%{optflags} ${extra_cflags[*]}" CXXFLAGS="%{optflags} ${extra_cfl
 export LDFLAGS="%{build_ldflags}"
 
 %{__python3} configure.py --prefix=%{_prefix} --verbose \
+           --enable-lto \
            --shared-openssl \
            --shared-zlib \
            --shared-brotli \
@@ -635,6 +637,8 @@ NODE_PATH=%{buildroot}%{_prefix}/lib/node_modules:%{buildroot}%{_prefix}/lib/nod
   Resolves: CVE-2023-30581 CVE-2023-30588 CVE-2023-30589 CVE-2023-30590
 - Replace /usr/etc/npmrc symlink with builtin configuration
   Resolves: rhbz#2222285
+- Enable LTO
+  Resolves: rhbz#1990096
 
 * Tue May 30 2023 Jan Staněk <jstanek@redhat.com> - 1:18.14.2-3
 - Update bundled c-ares to 1.19.1