Re: [Qemu-devel] [PATCH] tci: fix build breakage for target-sparc

[Stefan Weil <sw@weilnetz.de>]

Am 08.10.2012 22:45, schrieb Michael Roth:
> commit c28ae41 introduced GETPC() usage for sparc, which is currently
> not defined when building with --enable-tcg-interpreter. Add sparc to
> the list of targets we selectively define GETPC() for.
>
> Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
> ---
>   exec-all.h |    3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/exec-all.h b/exec-all.h
> index 6516da0..f7f649e 100644
> --- a/exec-all.h
> +++ b/exec-all.h
> @@ -296,7 +296,8 @@ extern int tb_invalidated_flag;
>   #if defined(CONFIG_TCG_INTERPRETER)
>   /* Alpha and SH4 user mode emulations and Softmmu call GETPC().
>      For all others, GETPC remains undefined (which makes TCI a little faster. */
> -# if defined(CONFIG_SOFTMMU) || defined(TARGET_ALPHA) || defined(TARGET_SH4)
> +# if defined(CONFIG_SOFTMMU) || defined(TARGET_ALPHA) || defined(TARGET_SH4) \
> +     || defined(TARGET_SPARC)
>   extern uintptr_t tci_tb_ptr;
>   #  define GETPC() tci_tb_ptr
>   # endif

Reviewed-by: Stefan Weil <sw@weilnetz.de>

Maybe whoever commits that patch can fix the comment, too.
I don't think we need a 2nd patch for that.

Regards

Stefan W.