Use show_debug_regs as a boolean
authorYao Qi <yao.qi@linaro.org>
Fri, 6 Mar 2015 14:14:27 +0000 (14:14 +0000)
committerYao Qi <yao.qi@linaro.org>
Fri, 6 Mar 2015 14:14:27 +0000 (14:14 +0000)
I happen to see that show_debug_regs is used as an arithmetic type,
but it should be a boolean,

  if (show_debug_regs > 1)

On the other hand, GDB RSP only allows setting it to either 0 or 1,
so it makes no sense to check whether it is greater than 1.  This
patch fixes it.

gdb/gdbserver:

2015-03-06  Yao Qi  <yao.qi@linaro.org>

* linux-aarch64-low.c (aarch64_insert_point): Use
show_debug_regs as a boolean.
(aarch64_remove_point): Likewise.

gdb/gdbserver/ChangeLog
gdb/gdbserver/linux-aarch64-low.c

index 7a99838ce876f3e545edbad7ab216931d7b97886..e735c8ef24674b5527ddf1cf27b3606f83143e39 100644 (file)
@@ -1,3 +1,9 @@
+2015-03-06  Yao Qi  <yao.qi@linaro.org>
+
+       * linux-aarch64-low.c (aarch64_insert_point): Use
+       show_debug_regs as a boolean.
+       (aarch64_remove_point): Likewise.
+
 2015-03-05  Pedro Alves  <palves@redhat.com>
 
        * lynx-low.c (lynx_target_ops): Install NULL hooks for
index a34fe5d1d8d690629cf8ae67e0565497329d7f1b..7934e78575f9dc15d58793249f7a6680d74af1cd 100644 (file)
@@ -990,7 +990,7 @@ aarch64_insert_point (enum raw_bkpt_type type, CORE_ADDR addr,
     ret =
       aarch64_handle_breakpoint (targ_type, addr, len, 1 /* is_insert */);
 
-  if (show_debug_regs > 1)
+  if (show_debug_regs)
     aarch64_show_debug_reg_state (aarch64_get_debug_reg_state (),
                                  "insert_point", addr, len, targ_type);
 
@@ -1027,7 +1027,7 @@ aarch64_remove_point (enum raw_bkpt_type type, CORE_ADDR addr,
     ret =
       aarch64_handle_breakpoint (targ_type, addr, len, 0 /* is_insert */);
 
-  if (show_debug_regs > 1)
+  if (show_debug_regs)
     aarch64_show_debug_reg_state (aarch64_get_debug_reg_state (),
                                  "remove_point", addr, len, targ_type);