Re: [PATCH 07/18] system: Remove unnecessary 'exec/translate-all.h' include

[Pierrick Bouvier]

On 12/12/24 10:53, Philippe Mathieu-Daudé wrote:
> At this point "exec/translate-all.h" only declare
> tb_check_watchpoint(), which isn't used by any of
> cpu-target.c or system/physmem.c, so remove its
> inclusion.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   cpu-target.c     | 1 -
>   system/physmem.c | 1 -
>   2 files changed, 2 deletions(-)
>
> diff --git a/cpu-target.c b/cpu-target.c
> index 5a7c3290814..4a88f1c6db8 100644
> --- a/cpu-target.c
> +++ b/cpu-target.c
> @@ -41,7 +41,6 @@
>   #include "exec/cpu-common.h"
>   #include "exec/exec-all.h"
>   #include "exec/tb-flush.h"
> -#include "exec/translate-all.h"
>   #include "exec/log.h"
>   #include "hw/core/accel-cpu.h"
>   #include "trace/trace-root.h"
> diff --git a/system/physmem.c b/system/physmem.c
> index 93c0ff60eb4..0fa6c37c774 100644
> --- a/system/physmem.c
> +++ b/system/physmem.c
> @@ -61,7 +61,6 @@
>   
>   #include "qemu/rcu_queue.h"
>   #include "qemu/main-loop.h"
> -#include "exec/translate-all.h"
>   #include "system/replay.h"
>   
>   #include "exec/memory-internal.h"

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>