fuzz: Split script formatting from script fuzz target #24349

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2202-fuzzFaster changing 3 files +31 −8

MarcoFalke commented at 11:27 AM on February 15, 2022: member
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
fuzz: Split script formatting from script fuzz target fae3f17823
MarcoFalke added the label Tests on Feb 15, 2022
MarcoFalke commented at 11:31 AM on February 15, 2022: member

Can be reviewed with --color-moved=dimmed-zebra
fanquake approved
fanquake commented at 4:48 PM on February 17, 2022: member

ACK fae3f178238df96554dc2495e040f5580b55408a
fanquake merged this on Feb 17, 2022
fanquake closed this on Feb 17, 2022
MarcoFalke deleted the branch on Feb 17, 2022
sidhujag referenced this in commit e0cac16f0b on Feb 18, 2022
DrahtBot locked this on Feb 17, 2023

Contributors

Labels