From: Andrew Burgess <andrew.burgess@embecosm.com>
Date: Sun, 8 Apr 2018 10:07:56 +0000 (+0100)
Subject: gdb/riscv: Use TYPE_SAFE_NAME
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=42ecac1732194293568ab6093c742ca74c1dbdf6;p=binutils-gdb.git

gdb/riscv: Use TYPE_SAFE_NAME

In debug printing, use TYPE_SAFE_NAME instead of replicating the
functionality using TYPE_NAME and a null check.

gdb/ChangeLog:

	* riscv-tdep.c (riscv_print_arg_location): Use TYPE_SAFE_NAME.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 25aded3c292..bae1d919877 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2018-07-10  Andrew Burgess  <andrew.burgess@embecosm.com>
+
+	* riscv-tdep.c (riscv_print_arg_location): Use TYPE_SAFE_NAME.
+
 2018-07-09  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* remote.c (show_hardware_watchpoint_limit): New function.
diff --git a/gdb/riscv-tdep.c b/gdb/riscv-tdep.c
index 4c68ef73fc5..e2be993eaf0 100644
--- a/gdb/riscv-tdep.c
+++ b/gdb/riscv-tdep.c
@@ -1939,12 +1939,8 @@ riscv_print_arg_location (ui_file *stream, struct gdbarch *gdbarch,
 			  struct riscv_arg_info *info,
 			  CORE_ADDR sp_refs, CORE_ADDR sp_args)
 {
-  const char* type_name = TYPE_NAME (info->type);
-  if (type_name == nullptr)
-    type_name = "???";
-
   fprintf_unfiltered (stream, "type: '%s', length: 0x%x, alignment: 0x%x",
-		      type_name, info->length, info->align);
+		      TYPE_SAFE_NAME (info->type), info->length, info->align);
   switch (info->argloc[0].loc_type)
     {
     case riscv_arg_info::location::in_reg: