Skip to content

Commit ab2f531

Browse files
committed
Merge bitcoin/bitcoin#28650: fuzz: Merge with -set_cover_merge=1
fa858d6 fuzz: Merge with -set_cover_merge=1 (MarcoFalke) Pull request description: This should be less controversial than commit 151a2b1. The overall size of the qa-assets repo is reduced further from 1.9GB to 1.6GB. Also, the runtime to iterate on the resulting folder is reduced further from ~1699s to ~1149s (N=1). ACKs for top commit: murchandamus: crACK fa858d6 dergoegge: ACK fa858d6 Tree-SHA512: e23fa93bd48f01d11c551b035004c678bd6d76bc24ac7d0d0a7883060804e6711763cbd0cd0ded3aad3e4c40da764decae81c2703388cc11961def3c89a4f9ba
2 parents cee39d0 + fa858d6 commit ab2f531

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

test/fuzz/test_runner.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,11 @@ def merge_inputs(*, fuzz_pool, corpus, test_list, src_dir, build_dir, merge_dir)
276276
for t in test_list:
277277
args = [
278278
os.path.join(build_dir, 'src', 'test', 'fuzz', 'fuzz'),
279-
'-merge=1',
279+
'-set_cover_merge=1',
280+
# set_cover_merge is used instead of -merge=1 to reduce the overall
281+
# size of the qa-assets git repository a bit, but more importantly,
282+
# to cut the runtime to iterate over all fuzz inputs [0].
283+
# [0] https://github.com/bitcoin-core/qa-assets/issues/130#issuecomment-1761760866
280284
'-shuffle=0',
281285
'-prefer_small=1',
282286
'-use_value_profile=0',

0 commit comments

Comments
 (0)