diff options
| author | merge-script <fanquake@gmail.com> | 2024-06-10 12:01:19 +0100 |
|---|---|---|
| committer | merge-script <fanquake@gmail.com> | 2024-06-10 12:01:19 +0100 |
| commit | cad127235e307d7de0e9cc04708dbd31aa6c24b0 (patch) | |
| tree | 2c9103c812a047a32d6cc816f99d398585d0226d /contrib/guix/manifest.scm | |
| parent | bd642ee15bda313bcf801cf63e4428c7a7d252c8 (diff) | |
| parent | fa780e1c25e8e98253e32d93db65f78a0092433f (diff) | |
| download | bitcoin-cad127235e307d7de0e9cc04708dbd31aa6c24b0.tar.xz | |
Merge bitcoin/bitcoin#30257: build: Remove --enable-gprof
fa780e1c25e8e98253e32d93db65f78a0092433f build: Remove --enable-gprof (MarcoFalke)
Pull request description:
It is unclear what benefit this option has, given that:
* `gprof` requires re-compilation (`perf` and other tools can instead be used on existing executables)
* `gprof` requires hardening to be disabled
* `gprof` doesn't work with `clang`
* `perf` is documented in the dev-notes, and test notes, and embedded into the functional test framework; `gprof` isn't
* Anyone really wanting to use it could pass the required flags to `./configure`
* I couldn't find any mention of the use of `gprof` in the discussions in this repo, apart from the initial pull request adding it (cfaac2a60f3ac63ae8deccb03d88bd559449b78c)
* Keeping it means that it needs to be maintained and ported to CMake
Fix all issues by removing it.
ACKs for top commit:
TheCharlatan:
ACK fa780e1c25e8e98253e32d93db65f78a0092433f
hebasto:
ACK fa780e1c25e8e98253e32d93db65f78a0092433f, I have reviewed the code and it looks OK.
willcl-ark:
crACK fa780e1c25e8e98253e32d93db65f78a0092433f
Tree-SHA512: 0a9ff363ac2bec8b743878a4e3147f18bc16823d00c5007568432c36320bd0199b13b6d0ce828a9a83c2cc434c058afaa64eb2eccfbd93ed85b81ce10c41760c
Diffstat (limited to 'contrib/guix/manifest.scm')
0 files changed, 0 insertions, 0 deletions