diff options
| author | MarcoFalke <[email protected]> | 2018-08-13 15:01:07 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2018-08-13 15:02:38 -0400 |
| commit | ddc3ec92b0e655a3da21ac2e85ec2e7ecb66c65b (patch) | |
| tree | e446883baf2424095ce0ad1a936d6d26c3a5cb9c /src/bitcoin-cli.cpp | |
| parent | Merge #13534: Don't assert(foo()) where foo() has side effects (diff) | |
| parent | ui: Compile boost:signals2 only once (diff) | |
| download | discoin-ddc3ec92b0e655a3da21ac2e85ec2e7ecb66c65b.tar.xz discoin-ddc3ec92b0e655a3da21ac2e85ec2e7ecb66c65b.zip | |
Merge #13634: ui: Compile boost::signals2 only once
fa5ce27385 ui: Compile boost:signals2 only once (MarcoFalke)
Pull request description:
ui is one of the modules that poison other modules with `boost/signals2` headers. This moves the include to the cpp file and uses a forward declaration in the header.
Locally this speeds up the incremental build (building everything that uses the ui module) with gcc by ~5% for me. Gcc uses ~5% less memory.
Would be nice if someone could verify the numbers roughly.
I presume the improvements will be more pronounced if the other models would stop exposing the boost header as well.
Tree-SHA512: 078360eba330ddbca4268bd8552927eae242a239e18dfded25ec20be72650a68cd83af7ac160690249b943d33ae35d15df1313f1f60a0c28b9526853aa7d1e40
Diffstat (limited to 'src/bitcoin-cli.cpp')
0 files changed, 0 insertions, 0 deletions