Hi, I wondered why changing something in "tcg/tcg.h" would trigger rebuilding the whole tree and figured the inclusion in "exec/cpu_ldst.h".
By making tlb_addr_write() static to accel/tcg/cputlb.c we can remove the "tcg/tcg.h" inclusion and reduce the number of objects to rebuild. I added tlb_assert_iotlb_entry_for_ptr_present() but there is this comment in target/arm/mte_helper.c which I don't understand much (so have no clue how to fix this TODO) but I suppose this would be to add a proper implementation and not need this ugly tlb_assert_iotlb_entry_for_ptr_present(): * TODO: Perhaps there should be a cputlb helper that returns a * matching tlb entry + iotlb entry. Regards, Phil. Philippe Mathieu-Daudé (6): target: Replace tcg_debug_assert() by assert() target/m68k: Include missing "tcg/tcg.h" header target/mips: Include missing "tcg/tcg.h" header accel/tcg: Include missing "tcg/tcg.h" header accel/tcg: Refactor debugging tlb_assert_iotlb_entry_for_ptr_present() exec/cpu_ldst: Move tlb* declarations to "exec/exec-all.h" include/exec/cpu_ldst.h | 52 ------------------------- include/exec/exec-all.h | 47 ++++++++++++++++++++++ target/arm/translate.h | 4 +- accel/tcg/cputlb.c | 23 +++++++++++ accel/tcg/tcg-accel-ops-mttcg.c | 1 + accel/tcg/tcg-accel-ops-rr.c | 1 + target/arm/mte_helper.c | 15 ++----- target/arm/sve_helper.c | 18 +++------ target/arm/translate-a64.c | 12 +++--- target/arm/translate-sve.c | 4 +- target/arm/translate.c | 36 ++++++++--------- target/hppa/translate.c | 4 +- target/m68k/op_helper.c | 1 + target/mips/msa_helper.c | 1 + target/rx/op_helper.c | 6 +-- target/rx/translate.c | 14 +++---- target/sh4/translate.c | 4 +- target/riscv/insn_trans/trans_rvv.c.inc | 2 +- 18 files changed, 127 insertions(+), 118 deletions(-) -- 2.26.2