This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 3/4] [ARC] Remove NPS-400 as a machine type
- From: Nick Clifton <nickc at redhat dot com>
- To: Graham Markall <graham dot markall at embecosm dot com>, binutils at sourceware dot org
- Cc: Claudiu dot Zissulescu at synopsys dot com
- Date: Mon, 20 Jun 2016 10:28:06 +0100
- Subject: Re: [PATCH 3/4] [ARC] Remove NPS-400 as a machine type
- Authentication-results: sourceware.org; auth=none
- References: <cover dot 1466085017 dot git dot graham dot markall at embecosm dot com> <1d8567f0a0634374af05de189e5d2c2c31d9b700 dot 1466085017 dot git dot graham dot markall at embecosm dot com>
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