On 9/21/21 18:28, Peter Maydell wrote:
Currently helper.c includes some code which is part of the arm
target's gdbstub support.  This code has a better home: in gdbstub.c
and gdbstub64.c.  Move it there.

Because aarch64_fpu_gdb_get_reg() and aarch64_fpu_gdb_set_reg() move
into gdbstub64.c, this means that they're now compiled only for
TARGET_AARCH64 rather than always.  That is the only case when they
would ever be used, but it does mean that the ifdef in
arm_cpu_register_gdb_regs_for_features() needs to be adjusted to
match.

Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
---
  target/arm/internals.h |   7 ++
  target/arm/gdbstub.c   | 130 ++++++++++++++++++++
  target/arm/gdbstub64.c | 140 +++++++++++++++++++++
  target/arm/helper.c    | 271 -----------------------------------------
  4 files changed, 277 insertions(+), 271 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

Reply via email to