Re: [PATCH 05/10] target/m68k/helper: Include missing "cpu.h" header

[Richard Henderson]

On 12/16/22 14:05, Philippe Mathieu-Daudé wrote:
> 'dh_ctype_fp' is defined as 'FPReg *', itself declared in "cpu.h".
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   target/m68k/helper.h | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/target/m68k/helper.h b/target/m68k/helper.h
> index c9bed2b884..aaf0e1668e 100644
> --- a/target/m68k/helper.h
> +++ b/target/m68k/helper.h
> @@ -1,3 +1,5 @@
> +#include "cpu.h"
> +
>   DEF_HELPER_1(bitrev, i32, i32)
>   DEF_HELPER_1(ff1, i32, i32)
>   DEF_HELPER_FLAGS_2(sats, TCG_CALL_NO_RWG_SE, i32, i32, i32)

No, see patch 1.

r~