diff options
| author | fanquake <fanquake@gmail.com> | 2022-02-17 16:48:33 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-02-17 16:48:44 +0000 |
| commit | edc0d327f1a07a00274066162871cc90f1220522 (patch) | |
| tree | 301e2d0bb141135ef766e75529e75c5114aa9878 /src/test/main.cpp | |
| parent | 003523d239aa41533482313aa3fd97c00641d69e (diff) | |
| parent | fae3f178238df96554dc2495e040f5580b55408a (diff) | |
Merge bitcoin/bitcoin#24349: fuzz: Split script formatting from script fuzz target
fae3f178238df96554dc2495e040f5580b55408a fuzz: Split script formatting from script fuzz target (MarcoFalke)
Pull request description:
This is a follow-up to commit 9237bdaac196951a437accaefa65638149b25978.
The target was improved a bit, but is still taking enormously long. See for example 4096 seconds in https://cirrus-ci.com/task/5153886888525824?logs=ci#L4451.
Most of the time is spent formatting the script. See the flamegraph: 
Thus, I suggest to split up the formatting into a new target. This will:
* Allow more fuzz cycles in the `script` target when exploring the search space with the fuzz engine
* Hopefully allow to reduce the fuzz inputs in `qa-assets` without losing coverage
ACKs for top commit:
fanquake:
ACK fae3f178238df96554dc2495e040f5580b55408a
Tree-SHA512: f86154b23019b7721e5dd10f54d11f4f7603d280471a396cb5256f4c460f48333318a60efe8b77fa8749a4abc67ad2631211b766fde5da70ded9fab8f904747b
Diffstat (limited to 'src/test/main.cpp')
0 files changed, 0 insertions, 0 deletions