Add EM_386/EM_IAMCU support to elfedit.c

binutils/

	* elfedit.c (enum elfclass): New.
	(input_elf_class): Change type to enum elfclass.
	(output_elf_class): New.
	(elf_class): Change return type to enum elfclass.  Support EM_386
	and EM_IAMCU.
	(update_elf_header): Check if input and output ELF classes match.
	(elf_machine): Support EM_386 and EM_IAMCU.
	(main): Update input_elf_class.  Set output_elf_class.
	* doc/binutils.texi: Update elfedit.

binutils/testsuite/

	* binutils-all/elfedit-5.d: New file.
	* binutils-all/elfedit.exp: Run elfedit-5.

binutils/ChangeLog

@@ -1,3 +1,15 @@
+2015-05-11  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* elfedit.c (enum elfclass): New.
+	(input_elf_class): Change type to enum elfclass.
+	(output_elf_class): New.
+	(elf_class): Change return type to enum elfclass.  Support EM_386
+	and EM_IAMCU.
+	(update_elf_header): Check if input and output ELF classes match.
+	(elf_machine): Support EM_386 and EM_IAMCU.
+	(main): Update input_elf_class.  Set output_elf_class.
+	* doc/binutils.texi: Update elfedit.
+
 2015-05-11  H.J. Lu  <hongjiu.lu@intel.com>
 
 	* dwarf.c (init_dwarf_regnames): Replace EM_486 with EM_IAMCU.

binutils/doc/binutils.texi

@@ -4614,8 +4614,8 @@ Set the matching input ELF machine type to @var{machine}.  If
 @option{--input-mach} isn't specified, it will match any ELF
 machine types.
 
-The supported ELF machine types are, @var{L1OM}, @var{K1OM} and
-@var{x86-64}.
+The supported ELF machine types are, @var{i386}, @var{IAMCU}, @var{L1OM},
+@var{K1OM} and @var{x86-64}.
 
 @item --output-mach=@var{machine}
 Change the ELF machine type in the ELF header to @var{machine}.  The

binutils/elfedit.c

@@ -54,7 +54,38 @@ static int input_elf_type = -1;
 static int output_elf_type = -1;
 static int input_elf_osabi = -1;
 static int output_elf_osabi = -1;
-static int input_elf_class = -1;
+enum elfclass
+  {
+    ELF_CLASS_UNKNOWN = -1,
+    ELF_CLASS_NONE = ELFCLASSNONE,
+    ELF_CLASS_32 = ELFCLASS32,
+    ELF_CLASS_64 = ELFCLASS64,
+    ELF_CLASS_BOTH
+  };
+static enum elfclass input_elf_class = ELF_CLASS_UNKNOWN;
+static enum elfclass output_elf_class = ELF_CLASS_BOTH;
+
+/* Return ELF class for a machine type, MACH.  */
+
+static enum elfclass
+elf_class (int mach)
+{
+  switch (mach)
+    {
+    case EM_386:
+    case EM_IAMCU:
+      return ELF_CLASS_32;
+    case EM_L1OM:
+    case EM_K1OM:
+      return ELF_CLASS_64;
+    case EM_X86_64:
+    case EM_NONE:
+      return ELF_CLASS_BOTH;
+    default:
+      error (_("Unknown machine type: %d\n"), mach);
+      return ELF_CLASS_UNKNOWN;
+    }
+}
 
 static int
 update_elf_header (const char *file_name, FILE *file)
@@ -86,17 +117,29 @@ update_elf_header (const char *file_name, FILE *file)
     return 1;
 
   class = elf_header.e_ident[EI_CLASS];
+  machine = elf_header.e_machine;
 
   /* Skip if class doesn't match. */
-  if (input_elf_class != -1 && class != input_elf_class)
+  if (input_elf_class == ELF_CLASS_UNKNOWN)
+    input_elf_class = elf_class (machine);
+
+  if (input_elf_class != ELF_CLASS_BOTH
+      && (int) input_elf_class != class)
     {
       error
-	(_("%s: Unmatched EI_CLASS: %d is not %d\n"),
+	(_("%s: Unmatched input EI_CLASS: %d is not %d\n"),
 	 file_name, class, input_elf_class);
       return 0;
     }
 
-  machine = elf_header.e_machine;
+  if (output_elf_class != ELF_CLASS_BOTH
+      && (int) output_elf_class != class)
+    {
+      error
+	(_("%s: Unmatched output EI_CLASS: %d is not %d\n"),
+	 file_name, class, output_elf_class);
+      return 0;
+    }
 
   /* Skip if e_machine doesn't match. */
   if (input_elf_machine != -1 && machine != input_elf_machine)
@@ -551,6 +594,10 @@ elf_osabi (const char *osabi)
 static int
 elf_machine (const char *mach)
 {
+  if (strcasecmp (mach, "i386") == 0)
+    return EM_386;
+  if (strcasecmp (mach, "iamcu") == 0)
+    return EM_IAMCU;
   if (strcasecmp (mach, "l1om") == 0)
     return EM_L1OM;
   if (strcasecmp (mach, "k1om") == 0)
@@ -567,25 +614,6 @@ elf_machine (const char *mach)
   return -1;
 }
 
-/* Return ELF class for a machine type, MACH.  */
-
-static int
-elf_class (int mach)
-{
-  switch (mach)
-    {
-    case EM_L1OM:
-    case EM_K1OM:
-    case EM_X86_64:
-      return ELFCLASS64;
-    case EM_NONE:
-      return ELFCLASSNONE;
-    default:
-      error (_("Unknown machine type: %d\n"), mach);
-      return -1;
-    }
-}
-
 /* Return ET_XXX for a type string, TYPE.  */
 
 static int
@@ -677,7 +705,7 @@ main (int argc, char ** argv)
 	  if (input_elf_machine < 0)
 	    return 1;
 	  input_elf_class = elf_class (input_elf_machine);
-	  if (input_elf_class < 0)
+	  if (input_elf_class == ELF_CLASS_UNKNOWN)
 	    return 1;
 	  break;
 
@@ -685,6 +713,9 @@ main (int argc, char ** argv)
 	  output_elf_machine = elf_machine (optarg);
 	  if (output_elf_machine < 0)
 	    return 1;
+	  output_elf_class = elf_class (output_elf_machine);
+	  if (output_elf_class == ELF_CLASS_UNKNOWN)
+	    return 1;
 	  break;
 
 	case OPTION_INPUT_TYPE:

binutils/testsuite/ChangeLog

@@ -1,3 +1,8 @@
+2015-05-11  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* binutils-all/elfedit-5.d: New file.
+	* binutils-all/elfedit.exp: Run elfedit-5.
+
 2015-04-30  Nick Clifton  <nickc@redhat.com>
 
 	* binutils-all/objdump.exp (cpus_expected): Add MeP CPU names.

binutils/testsuite/binutils-all/elfedit-5.d

@@ -0,0 +1,17 @@
+#PROG: elfedit
+#elfedit: --output-mach iamcu
+#source: empty.s
+#as: --32
+#readelf: -h
+#name: Update ELF header 5
+#target: x86_64-*-* i386-*-*
+
+#...
+ELF Header:
+  Magic:   7f 45 4c 46 01 01 01 00 00 00 00 00 00 00 00 00 
+  Class:                             ELF32
+  Data:                              2's complement, little endian
+  Version:                           1 \(current\)
+#...
+  Machine:                           Intel MCU
+#...

binutils/testsuite/binutils-all/elfedit.exp

@@ -32,3 +32,4 @@ run_dump_test "elfedit-1"
 run_dump_test "elfedit-2"
 run_dump_test "elfedit-3"
 run_dump_test "elfedit-4"
+run_dump_test "elfedit-5"