exec: Move translation declarations to 'translate-all.h'

[PATCH 02/13] exec/cpu_ldst: Include 'cpu.h' to get target_ulong definition

Posted by Philippe Mathieu-Daudé via 4 years ago

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 include/exec/cpu_ldst.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/exec/cpu_ldst.h b/include/exec/cpu_ldst.h
index a878fd0105..5c999966de 100644
--- a/include/exec/cpu_ldst.h
+++ b/include/exec/cpu_ldst.h
@@ -64,6 +64,7 @@
 
 #include "exec/memopidx.h"
 #include "qemu/int128.h"
+#include "cpu.h"
 
 #if defined(CONFIG_USER_ONLY)
 /* sparc32plus has 64bit long but 32bit space address
-- 
2.34.1

Re: [PATCH 02/13] exec/cpu_ldst: Include 'cpu.h' to get target_ulong definition

Posted by Richard Henderson 4 years ago

On 2/9/22 02:22, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>   include/exec/cpu_ldst.h | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/include/exec/cpu_ldst.h b/include/exec/cpu_ldst.h
> index a878fd0105..5c999966de 100644
> --- a/include/exec/cpu_ldst.h
> +++ b/include/exec/cpu_ldst.h
> @@ -64,6 +64,7 @@
>   
>   #include "exec/memopidx.h"
>   #include "qemu/int128.h"
> +#include "cpu.h"

I guess this is about as good as we can do for now.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~