From a16bc3f36c7ab9fcd758f22544cf46561402b1f9 Mon Sep 17 00:00:00 2001 From: Sam James Date: Sun, 8 Jun 2025 13:06:54 +0100 Subject: [PATCH] instrumentation: drop removed TODO flag from GCC plugins TODO_verify_il was removed in GCC trunk by 9739ae9384dd7cd3bb1c7683d6b80b7a9116eaf8, so drop it here. --- instrumentation/afl-gcc-cmplog-pass.so.cc | 3 +-- instrumentation/afl-gcc-cmptrs-pass.so.cc | 3 +-- instrumentation/afl-gcc-pass.so.cc | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/instrumentation/afl-gcc-cmplog-pass.so.cc b/instrumentation/afl-gcc-cmplog-pass.so.cc index 38fce961..f3906719 100644 --- a/instrumentation/afl-gcc-cmplog-pass.so.cc +++ b/instrumentation/afl-gcc-cmplog-pass.so.cc @@ -44,8 +44,7 @@ static const struct pass_data afl_cmplog_pass_data = { .properties_provided = 0, .properties_destroyed = 0, .todo_flags_start = 0, - .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg | TODO_verify_il | - TODO_rebuild_cgraph_edges), + .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg | TODO_rebuild_cgraph_edges), }; diff --git a/instrumentation/afl-gcc-cmptrs-pass.so.cc b/instrumentation/afl-gcc-cmptrs-pass.so.cc index 360b035a..94a5599c 100644 --- a/instrumentation/afl-gcc-cmptrs-pass.so.cc +++ b/instrumentation/afl-gcc-cmptrs-pass.so.cc @@ -44,8 +44,7 @@ static const struct pass_data afl_cmptrs_pass_data = { .properties_provided = 0, .properties_destroyed = 0, .todo_flags_start = 0, - .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg | TODO_verify_il | - TODO_rebuild_cgraph_edges), + .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg | TODO_rebuild_cgraph_edges), }; diff --git a/instrumentation/afl-gcc-pass.so.cc b/instrumentation/afl-gcc-pass.so.cc index eeef4e02..f7ec3df5 100644 --- a/instrumentation/afl-gcc-pass.so.cc +++ b/instrumentation/afl-gcc-pass.so.cc @@ -65,7 +65,6 @@ The new pass is to be a GIMPLE_PASS. Given the sort of instrumentation it's supposed to do, its todo_flags_finish will certainly need TODO_update_ssa, and TODO_cleanup_cfg. - TODO_verify_il is probably desirable, at least during debugging. TODO_rebuild_cgraph_edges is required only in the out-of-line instrumentation mode. @@ -148,7 +147,7 @@ static constexpr struct pass_data afl_pass_data = { .properties_provided = 0, .properties_destroyed = 0, .todo_flags_start = 0, - .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg | TODO_verify_il), + .todo_flags_finish = (TODO_update_ssa | TODO_cleanup_cfg), };