[PATCH] S/390: Add -Mesa/-Mzarch disassembler options.

["Andreas Krebbel" <Andreas dot Krebbel at de dot ibm dot com>]

Hello,

currently objdump prints only esa mode instructions for 31 bit
executables.  But such executables might also contain zarch
instructions if compiled with -m31 -mzarch.

The attached patch adds two S/390 specific disassembler options -Mesa
and -Mzarch which allow to override the decision normally based on the
ABI of binary.

Tested on s390x and s390.

Ok to apply?

Bye,

-Andreas-


2008-04-10  Andreas Krebbel  <krebbel1@de.ibm.com>

	* s390-dis.c (init_disasm): Evaluate disassembler_options.
	(print_s390_disassembler_options): New function.
	Include opintl.h.
	* disassemble.c (disassembler_usage): Invoke
	print_s390_disassembler_options.

2008-04-10  Andreas Krebbel  <krebbel1@de.ibm.com>

	* include/dis-asm.h (print_s390_disassembler_options):
	Prototype added.


diff --exclude=.pc -N -p -u -r src2/include/dis-asm.h src/include/dis-asm.h
--- src2/include/dis-asm.h	2008-04-10 15:17:04.000000000 +0200
+++ src/include/dis-asm.h	2008-04-10 15:08:13.000000000 +0200
@@ -291,6 +291,7 @@ extern void print_mips_disassembler_opti
 extern void print_ppc_disassembler_options (FILE *);
 extern void print_arm_disassembler_options (FILE *);
 extern void parse_arm_disassembler_option (char *);
+extern void print_s390_disassembler_options (FILE *);
 extern int  get_arm_regname_num_options (void);
 extern int  set_arm_regname_option (int);
 extern int  get_arm_regnames (int, const char **, const char **, const char *const **);
diff --exclude=.pc -N -p -u -r src2/opcodes/disassemble.c src/opcodes/disassemble.c
--- src2/opcodes/disassemble.c	2008-04-10 15:17:04.000000000 +0200
+++ src/opcodes/disassemble.c	2008-04-10 15:08:13.000000000 +0200
@@ -466,6 +466,9 @@ disassembler_usage (stream)
 #ifdef ARCH_i386
   print_i386_disassembler_options (stream);
 #endif
+#ifdef ARCH_s390
+  print_s390_disassembler_options (stream);
+#endif
 
   return;
 }
diff --exclude=.pc -N -p -u -r src2/opcodes/s390-dis.c src/opcodes/s390-dis.c
--- src2/opcodes/s390-dis.c	2008-04-10 15:17:04.000000000 +0200
+++ src/opcodes/s390-dis.c	2008-04-10 15:08:13.000000000 +0200
@@ -23,6 +23,7 @@
 #include "ansidecl.h"
 #include "sysdep.h"
 #include "dis-asm.h"
+#include "opintl.h"
 #include "opcode/s390.h"
 
 static int init_flag = 0;
@@ -36,6 +37,7 @@ init_disasm (struct disassemble_info *in
 {
   const struct s390_opcode *opcode;
   const struct s390_opcode *opcode_end;
+  const char *p;
 
   memset (opc_index, 0, sizeof (opc_index));
   opcode_end = s390_opcodes + s390_num_opcodes;
@@ -46,17 +48,34 @@ init_disasm (struct disassemble_info *in
 	     (opcode[1].opcode[0] == opcode->opcode[0]))
 	opcode++;
     }
-  switch (info->mach)
+
+  for (p = info->disassembler_options; p != NULL; )
     {
-    case bfd_mach_s390_31:
-      current_arch_mask = 1 << S390_OPCODE_ESA;
-      break;
-    case bfd_mach_s390_64:
-      current_arch_mask = 1 << S390_OPCODE_ZARCH;
-      break;
-    default:
-      abort ();
+      if (CONST_STRNEQ (p, "esa"))
+	current_arch_mask = 1 << S390_OPCODE_ESA;
+      else if (CONST_STRNEQ (p, "zarch"))
+	current_arch_mask = 1 << S390_OPCODE_ZARCH;
+      else
+	fprintf (stderr, "Unknown S/390 disassembler option: %s\n", p);
+
+      p = strchr (p, ',');
+      if (p != NULL)
+	p++;
     }
+
+  if (!current_arch_mask)
+    switch (info->mach)
+      {
+      case bfd_mach_s390_31:
+	current_arch_mask = 1 << S390_OPCODE_ESA;
+	break;
+      case bfd_mach_s390_64:
+	current_arch_mask = 1 << S390_OPCODE_ZARCH;
+	break;
+      default:
+	abort ();
+      }
+
   init_flag = 1;
 }
 
@@ -250,3 +269,14 @@ print_insn_s390 (bfd_vma memaddr, struct
       return 1;
     }
 }
+
+void
+print_s390_disassembler_options (FILE *stream)
+{
+  fprintf (stream, _("\n\
+The following S/390 specific disassembler options are supported for use\n\
+with the -M switch (multiple options should be separated by commas):\n"));
+
+  fprintf (stream, _("  esa         Disassemble in ESA architecture mode\n"));
+  fprintf (stream, _("  zarch       Disassemble in z/Architecture mode\n"));
+}