gdb: Use passed gdbarch instead of calling current_inferior

In do_examine function, use passed gdbarch when checking if an address
is tagged instead of calling current_inferior()->arch() to make the code
more localized and help modularity by not calling a current_* function,
which disguises the use of a global state/variable. There is no change
in the code behavior.

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Suggested-by: Thiago Jung Bauermann <thiago.bauermann@linaro.org>
Approved-By: Luis Machado <luis.machado@arm.com>
Tested-By: Luis Machado <luis.machado@arm.com>

diff --git a/gdb/printcmd.c b/gdb/printcmd.c
index 5635f60531419c974ec884153867e0687ddc1256..4edbd458e4d48c545b9fce57c54aff518fb8dd9a 100644 (file)
--- a/gdb/printcmd.c
+++ b/gdb/printcmd.c
@@ -1132,7 +1132,7 @@ do_examine (struct format_data fmt, struct gdbarch *gdbarch, CORE_ADDR addr)
            = value_from_ulongest (builtin_type (gdbarch)->builtin_data_ptr,
                                   tag_laddr);
 
-         if (gdbarch_tagged_address_p (current_inferior ()->arch  (), v_addr))
+         if (gdbarch_tagged_address_p (gdbarch, v_addr))
            {
              /* Fetch the allocation tag.  */
              struct value *tag