Changeset View
Changeset View
Standalone View
Standalone View
src/test/fuzz/CMakeLists.txt
Show All 29 Lines | foreach(_fuzz_test_name ${ARGN}) | ||||
) | ) | ||||
sanitize_c_cxx_definition("" ${_fuzz_test_name} _target_definition) | sanitize_c_cxx_definition("" ${_fuzz_test_name} _target_definition) | ||||
string(TOUPPER ${_target_definition} _target_definition) | string(TOUPPER ${_target_definition} _target_definition) | ||||
target_compile_definitions(${_fuzz_target_name} PRIVATE ${_target_definition}) | target_compile_definitions(${_fuzz_target_name} PRIVATE ${_target_definition}) | ||||
endforeach() | endforeach() | ||||
endfunction() | endfunction() | ||||
function(add_regular_fuzz_targets) | |||||
foreach(_fuzz_test_name ${ARGN}) | |||||
sanitize_target_name("fuzz-" ${_fuzz_test_name} _fuzz_target_name) | |||||
add_fuzz_target( | |||||
${_fuzz_target_name} | |||||
${_fuzz_test_name} | |||||
# Sources | |||||
"${_fuzz_test_name}.cpp" | |||||
) | |||||
endforeach() | |||||
endfunction() | |||||
add_regular_fuzz_targets( | |||||
transaction | |||||
) | |||||
Fabien: I'm not a fan of having a different function for each target.
You can either use… | |||||
add_deserialize_fuzz_targets( | add_deserialize_fuzz_targets( | ||||
address_deserialize | address_deserialize | ||||
addrman_deserialize | addrman_deserialize | ||||
banentry_deserialize | banentry_deserialize | ||||
block_deserialize | block_deserialize | ||||
blockheader_deserialize | blockheader_deserialize | ||||
blocklocator_deserialize | blocklocator_deserialize | ||||
blockmerkleroot | blockmerkleroot | ||||
blocktransactions_deserialize | blocktransactions_deserialize | ||||
blocktransactionsrequest_deserialize | blocktransactionsrequest_deserialize | ||||
blockundo_deserialize | blockundo_deserialize | ||||
bloomfilter_deserialize | bloomfilter_deserialize | ||||
coins_deserialize | coins_deserialize | ||||
diskblockindex_deserialize | diskblockindex_deserialize | ||||
inv_deserialize | inv_deserialize | ||||
messageheader_deserialize | messageheader_deserialize | ||||
netaddr_deserialize | netaddr_deserialize | ||||
service_deserialize | service_deserialize | ||||
transaction_deserialize | |||||
txoutcompressor_deserialize | txoutcompressor_deserialize | ||||
txundo_deserialize | txundo_deserialize | ||||
) | ) | ||||
add_fuzz_target( | add_fuzz_target( | ||||
fuzz-script_flags | fuzz-script_flags | ||||
script_flags | script_flags | ||||
# Sources | # Sources | ||||
fuzz.cpp | fuzz.cpp | ||||
script_flags.cpp | script_flags.cpp | ||||
) | ) |
I'm not a fan of having a different function for each target.
You can either use add_fuzz_target directly (like the fuzz-script_flags target does) or make the fuzz-script_flags target use your new function if you need it for later diffs, keeping arg_fuzz_target private.