gdb: turn arc_debug into a bool

Shahab suggested we get rid of the verbosity level for the ARC debug
logging [1].  This patch does that, before doing any other change.

gdb/ChangeLog:

	* arc-tdep.h (arc_debug): Change type to bool.
	* arc-tdep.c (arc_debug): Change type to bool.
	(arc_analyze_prologue): Adjust.
	(_initialize_arc_tdep): Use add_setshow_boolean_cmd.
	* arc-linux-nat.c (ps_get_thread_area): Adjust.

[1] https://sourceware.org/pipermail/gdb-patches/2021-January/175075.html

Change-Id: I16688bd42ed8978ae1acf57012c8d41a943044a5
This commit is contained in:
Simon Marchi
2021-01-13 14:32:23 -05:00
parent 506195754c
commit fb0f5031bb
4 changed files with 18 additions and 10 deletions

View File

@ -1,3 +1,11 @@
2021-01-13 Simon Marchi <simon.marchi@polymtl.ca>
* arc-tdep.h (arc_debug): Change type to bool.
* arc-tdep.c (arc_debug): Change type to bool.
(arc_analyze_prologue): Adjust.
(_initialize_arc_tdep): Use add_setshow_boolean_cmd.
* arc-linux-nat.c (ps_get_thread_area): Adjust.
2021-01-13 Simon Marchi <simon.marchi@polymtl.ca> 2021-01-13 Simon Marchi <simon.marchi@polymtl.ca>
* auto-load.c (auto_load_objfile_script_1): Use bool. * auto-load.c (auto_load_objfile_script_1): Use bool.

View File

@ -295,7 +295,7 @@ ps_err_e
ps_get_thread_area (struct ps_prochandle *ph, lwpid_t lwpid, int idx, ps_get_thread_area (struct ps_prochandle *ph, lwpid_t lwpid, int idx,
void **base) void **base)
{ {
if (arc_debug >= 2) if (arc_debug)
debug_printf ("arc-linux-nat: ps_get_thread_area called"); debug_printf ("arc-linux-nat: ps_get_thread_area called");
if (ptrace (PTRACE_GET_THREAD_AREA, lwpid, NULL, base) != 0) if (ptrace (PTRACE_GET_THREAD_AREA, lwpid, NULL, base) != 0)

View File

@ -87,7 +87,7 @@ struct arc_frame_cache
/* Global debug flag. */ /* Global debug flag. */
int arc_debug; bool arc_debug;
/* List of "maintenance print arc" commands. */ /* List of "maintenance print arc" commands. */
@ -1421,7 +1421,7 @@ arc_analyze_prologue (struct gdbarch *gdbarch, const CORE_ADDR entrypoint,
arc_insn_decode (current_prologue_end, &di, arc_delayed_print_insn, arc_insn_decode (current_prologue_end, &di, arc_delayed_print_insn,
&insn); &insn);
if (arc_debug >= 2) if (arc_debug)
arc_insn_dump (insn); arc_insn_dump (insn);
/* If this instruction is in the prologue, fields in the cache will be /* If this instruction is in the prologue, fields in the cache will be
@ -2517,10 +2517,10 @@ _initialize_arc_tdep ()
&maintenance_print_arc_list); &maintenance_print_arc_list);
/* Debug internals for ARC GDB. */ /* Debug internals for ARC GDB. */
add_setshow_zinteger_cmd ("arc", class_maintenance, add_setshow_boolean_cmd ("arc", class_maintenance,
&arc_debug, &arc_debug,
_("Set ARC specific debugging."), _("Set ARC specific debugging."),
_("Show ARC specific debugging."), _("Show ARC specific debugging."),
_("Non-zero enables ARC specific debugging."), _("When set, ARC specific debugging is enabled."),
NULL, NULL, &setdebuglist, &showdebuglist); NULL, NULL, &setdebuglist, &showdebuglist);
} }

View File

@ -112,7 +112,7 @@ enum arc_regnum
#define arc_print(fmt, args...) fprintf_unfiltered (gdb_stdlog, fmt, ##args) #define arc_print(fmt, args...) fprintf_unfiltered (gdb_stdlog, fmt, ##args)
extern int arc_debug; extern bool arc_debug;
/* Target-dependent information. */ /* Target-dependent information. */