diff options
| author | fanquake <fanquake@gmail.com> | 2023-12-06 12:26:02 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2024-01-05 15:17:50 +0000 |
| commit | 2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e (patch) | |
| tree | a04421a85baab197ecfb0fcee5073adacc03bb68 /depends/hosts/mingw32.mk | |
| parent | c80f57ba575af96890f185765a53a62ef58ef2c8 (diff) | |
| download | bitcoin-2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e.tar.xz | |
build: remove --enable-lto
This has outlived its usefulness, doesn't gel well with
newer compilers & `-flto` related options, i.e thin vs full, or `=auto`,
and having `-flto` as the only option means that sometimes this just
needs to be worked around, i.e in oss-fuzz:
https://github.com/google/oss-fuzz/blob/master/projects/bitcoin-core/build.sh.
While it was convenient when `-flto` was newer, support for `-flto` is now
in all compilers we use, and there's also no-longer any real need
for us to treat `-flto` different to any other optimization option.
Remove it, to remove build complexity, and so there's no need
to port a similar option to CMake.
Note that the LTO option remains in depends, because we still a way to
build packages that have LTO specific patches/options.
If we decide to merge this, I'll follow up downstream in oss-fuzz first,
to make sure we don't break the build.
Diffstat (limited to 'depends/hosts/mingw32.mk')
| -rw-r--r-- | depends/hosts/mingw32.mk | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/depends/hosts/mingw32.mk b/depends/hosts/mingw32.mk index fc1cc1afbe..15aa7cd25a 100644 --- a/depends/hosts/mingw32.mk +++ b/depends/hosts/mingw32.mk @@ -6,10 +6,6 @@ mingw32_CFLAGS=-pipe -std=$(C_STANDARD) mingw32_CXXFLAGS=-pipe -std=$(CXX_STANDARD) ifneq ($(LTO),) -mingw32_CFLAGS += -flto -mingw32_CXXFLAGS += -flto -mingw32_LDFLAGS += -flto - mingw32_AR = $(host_toolchain)gcc-ar mingw32_NM = $(host_toolchain)gcc-nm mingw32_RANLIB = $(host_toolchain)gcc-ranlib |