diff options
| author | fanquake <fanquake@gmail.com> | 2024-01-16 09:38:17 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2024-01-16 09:42:12 +0000 |
| commit | 2ac2821a74efdd0f61f091b0fc774cc386930c95 (patch) | |
| tree | 621a1ec5943a0a13c4f5809ac1c9c1ccbbdb201c /ci | |
| parent | 05c4c5a4347018d6d0e81b8f12cf5bdcaa8b4a89 (diff) | |
| parent | 2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e (diff) | |
| download | bitcoin-2ac2821a74efdd0f61f091b0fc774cc386930c95.tar.xz | |
Merge bitcoin/bitcoin#29185: build: remove `--enable-lto`
2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e build: remove --enable-lto (fanquake)
Pull request description:
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.
ACKs for top commit:
TheCharlatan:
ACK 2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e
hebasto:
ACK 2d1b1c7daeeada3f737e62ceb2db7484cde5ff4e.
Tree-SHA512: 91812de7da35346f51850714a188fcffbac478bc8b348bf756c2555fcbde86ba622ac2fb77d294dea0378c741d3656f06121ef3a795aeed63fd170fc31bfa5af
Diffstat (limited to 'ci')
0 files changed, 0 insertions, 0 deletions