[PATCH 3/4] [ARC] Remove NPS-400 as a machine type

Nick Clifton nickc@redhat.com
Mon Jun 20 09:28:00 GMT 2016


Hi Graham,

> bfd/ChangeLog:
> 
>         * archures.c: Remove bfd_mach_arc_nps400.
>         * bfd-in2.h: Likewise.
>         * cpu-arc.c (arch_info_struct): Likewise.
>         * elf32-arc.c (arc_elf_object_p, arc_elf_final_write_processing):
>         Likewise.
> 
> binutils/ChangeLog:
> 
>         * readelf.c (decode_ARC_machine_flags): Remove E_ARC_MACH_NPS400 case.
> 
> gas/ChangeLog:
> 
>         * config/tc-arc.c (cpu_type): Remove nps400 CPU type entry
>         * config/tc-arc.c (check_zol): Remove bfd_mach_arc_nps400 case.
> 
> include/ChangeLog:
> 
>         * elf/arc.h: Remove E_ARC_MACH_NPS400
>         * opcode/arc.h: Remove ARC_OPCODE_NPS400
> 
> opcodes/ChangeLog:
> 
>         * arc-dis.c (arc_insn_length, print_insn_arc): Remove
>         bfd_mach_arc_nps400.

I am not sure that this patch is a good idea.  What happens if the tools
(readelf, objcopy, objdump, etc) are used on an old binary that still has
the old E_ARC_MACH_NPS400 value set in the ELF flag bits ?  Presumably they
won't recognise it.  If there is a possibility that these old binaries exist
then we must maintain backwards compatibility for them.

Cheers
  Nick


 



More information about the Binutils mailing list