Re: [Qemu-devel] [PATCH 2/2] disas: Remove uses of CPU env

[Richard Henderson]

On 05/17/2015 12:51 PM, Peter Crosthwaite wrote:
> disas does not need to access the CPU env for any reason. Change the
> APIs to accept CPU pointers instead. Small change pattern needs to be
> applied to all target translate.c. This brings us closer to making
> disas.o a common-obj and less architecture specific in general.
> 
> Cc: Richard Henderson <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Cc: "Edgar E. Iglesias" <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Eduardo Habkost <address@hidden>
> Cc: Michael Walle <address@hidden>
> Cc: Aurelien Jarno <address@hidden>
> Cc: Leon Alrae <address@hidden>
> Cc: Jia Liu <address@hidden>
> Cc: Alexander Graf <address@hidden>
> Cc: Mark Cave-Ayland <address@hidden>
> Cc: Bastian Koppelmann <address@hidden>
> Cc: Guan Xuetao <address@hidden>
> Cc: Max Filippov <address@hidden>
> Signed-off-by: Peter Crosthwaite <address@hidden>
> ---
>  disas.c                       | 14 +++++++-------
>  include/disas/disas.h         |  4 ++--
>  include/qemu/log.h            |  4 ++--
>  monitor.c                     |  2 +-
>  target-alpha/translate.c      |  2 +-
>  target-arm/translate-a64.c    |  2 +-
>  target-arm/translate.c        |  2 +-
>  target-cris/translate.c       |  2 +-
>  target-i386/translate.c       |  2 +-
>  target-lm32/translate.c       |  2 +-
>  target-m68k/translate.c       |  2 +-
>  target-microblaze/translate.c |  2 +-
>  target-mips/translate.c       |  2 +-
>  target-openrisc/translate.c   |  2 +-
>  target-ppc/translate.c        |  2 +-
>  target-s390x/translate.c      |  2 +-
>  target-sh4/translate.c        |  2 +-
>  target-sparc/translate.c      |  2 +-
>  target-tricore/translate.c    |  2 +-
>  target-unicore32/translate.c  |  2 +-
>  target-xtensa/translate.c     |  2 +-
>  21 files changed, 29 insertions(+), 29 deletions(-)

Reviewed-by: Richard Henderson <address@hidden>


r~