public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [COMMITTED] opcodes: xtensa: implement styled disassembly
@ 2023-01-03 21:57 Max Filippov
  0 siblings, 0 replies; only message in thread
From: Max Filippov @ 2023-01-03 21:57 UTC (permalink / raw)
  To: binutils; +Cc: Max Filippov

opcodes/
	* xtensa-dis.c (print_xtensa_operand)
	(print_insn_xtensa): Replace fprintf_func with
	fprintf_styled_func.
---
 opcodes/xtensa-dis.c | 33 ++++++++++++++++++++++-----------
 1 file changed, 22 insertions(+), 11 deletions(-)

diff --git a/opcodes/xtensa-dis.c b/opcodes/xtensa-dis.c
index 8a26fd7c1239..3631f7fad715 100644
--- a/opcodes/xtensa-dis.c
+++ b/opcodes/xtensa-dis.c
@@ -238,24 +238,28 @@ print_xtensa_operand (bfd_vma memaddr,
       else
 	{
 	  if ((signed_operand_val > -256) && (signed_operand_val < 256))
-	    (*info->fprintf_func) (info->stream, "%d", signed_operand_val);
+	    (*info->fprintf_styled_func) (info->stream, dis_style_immediate,
+					  "%d", signed_operand_val);
 	  else
-	    (*info->fprintf_func) (info->stream, "0x%x", signed_operand_val);
+	    (*info->fprintf_styled_func) (info->stream, dis_style_immediate,
+					  "0x%x", signed_operand_val);
 	}
     }
   else
     {
       int i = 1;
       xtensa_regfile opnd_rf = xtensa_operand_regfile (isa, opc, opnd);
-      (*info->fprintf_func) (info->stream, "%s%u",
-			     xtensa_regfile_shortname (isa, opnd_rf),
-			     operand_val);
+      (*info->fprintf_styled_func) (info->stream, dis_style_register,
+				    "%s%u",
+				    xtensa_regfile_shortname (isa, opnd_rf),
+				    operand_val);
       while (i < xtensa_operand_num_regs (isa, opc, opnd))
 	{
 	  operand_val++;
-	  (*info->fprintf_func) (info->stream, ":%s%u",
-				 xtensa_regfile_shortname (isa, opnd_rf),
-				 operand_val);
+	  (*info->fprintf_styled_func) (info->stream, dis_style_register,
+					":%s%u",
+					xtensa_regfile_shortname (isa, opnd_rf),
+					operand_val);
 	  i++;
 	}
     }
@@ -404,7 +408,13 @@ print_insn_xtensa (bfd_vma memaddr, struct disassemble_info *info)
 	}
       else
 	{
-	  (*info->fprintf_func) (info->stream, ".byte %#02x", priv.byte_buf[0]);
+	  (*info->fprintf_styled_func) (info->stream,
+					dis_style_assembler_directive,
+					".byte");
+	  (*info->fprintf_func) (info->stream, "\t");
+	  (*info->fprintf_styled_func) (info->stream,
+					dis_style_immediate,
+					"%#02x", priv.byte_buf[0]);
 	  return 1;
 	}
     }
@@ -425,8 +435,9 @@ print_insn_xtensa (bfd_vma memaddr, struct disassemble_info *info)
 
       xtensa_format_get_slot (isa, fmt, n, insn_buffer, slot_buffer);
       opc = xtensa_opcode_decode (isa, fmt, n, slot_buffer);
-      (*info->fprintf_func) (info->stream, "%s",
-			     xtensa_opcode_name (isa, opc));
+      (*info->fprintf_styled_func) (info->stream,
+				    dis_style_mnemonic, "%s",
+				    xtensa_opcode_name (isa, opc));
 
       if (xtensa_opcode_is_branch (isa, opc))
 	info->insn_type = dis_condbranch;
-- 
2.30.2


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-01-03 21:57 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-03 21:57 [COMMITTED] opcodes: xtensa: implement styled disassembly Max Filippov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).