Re: New port: CRX

["Tomer Levi" <Tomer dot Levi at nsc dot com>]

Thanks, Nick.

These are indeed great news for us.
The lacking features you mentioned are on my list, and will be dealt in 
the near future.

Regards,
      Tomer Levi

Software Engineer                                   phone: 
+49-8141-35-1378
CompactRISC Development Tools  fax:        +49-8141-35-11-1378
National Semiconductor GmbH         mailto:Tomer.Levi@nsc.com
Livry-Gargan Str. 10                                internet: 
http://www.national.com
82256 Fuerstenfeldbruck, Germany





"Nick Clifton" <nickc@redhat.com>
07/07/2004 07:48 PM
 
        To:     "Tomer Levi" <Tomer.Levi@nsc.com>
        cc:     binutils@sources.redhat.com
        Subject:        Re: New port: CRX


Hi Tomer,


> Following our correspondence, below you can find the Changelogs for each 

> subdirectory in the Binutils project.

Thanks very much.  I have applied your patches and checked in the 
results.  Apart from some minor formatting tidy-ups the only real change 
I made was to the elf32_crx_gc_mark_hook() function -  it appeared to 
have an unnecessary switch statement within it.

There are a couple of features which I hope that you will be submitting 
in the future:

   * A CRX specific section to the GAS testsuite.

   * Support for the VTABLE_INHERIT and VTABLE_ENTRY relocs used by G++. 
  (Lack of this support is causing 2 unexpected failures in the GAS 
testsuite).

   * Support in GAS for evaluating the difference between two symbols 
which are defined in the same section.  (Lack of this support is causing 
3 unexpected failures in the GAS testsuite).

Please note that you will need to submit your patch to the top level 
configure files (config.sub, configure.in) to a separate mailing list:

     config-patches@gnu.org

Cheers
   Nick


> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
> 
>          * Makefile.am (ALL_MACHINES): Add cpu-crx.lo.
>          (ALL_MACHINES_CFILES): Add cpu-crx.c.
>          (BFD32_BACKENDS): Add elf32-crx.lo.
>          (BFD32_BACKENDS_CFILES): Add elf32-crx.c.
>          (cpu-crx.lo): New target.
>          (elf32-crx.lo): New target.
>          * Makefile.in: Regenerate.
>          * archures.c (bfd_architecture): Add bfd_{arch,mach}_crx.
>          (bfd_archures_list): Add bfd_crx_arch.
>          * bfd-in2.h: Regenerate.
>          * config.bfd: Handle crx-*-elf*, crx*.
>          * configure.in: Handle bfd_elf32_crx_vec.
>          * configure: Regenerate.
>          * cpu-crx.c: New file.
>          * elf32-crx.c: Likewise.
>          * libbfd.h: Regenerate.
>          * reloc.c: Add BFD_RELOC_CRX_REL4, BFD_RELOC_CRX_REL8, 
>          BFD_RELOC_CRX_REL8_CMP, BFD_RELOC_CRX_REL16, 
BFD_RELOC_CRX_REL24, 
>          BFD_RELOC_CRX_REL32, BFD_RELOC_CRX_REGREL12, 
> BFD_RELOC_CRX_REGREL22, 
>          BFD_RELOC_CRX_REGREL28, BFD_RELOC_CRX_REGREL32, 
> BFD_RELOC_CRX_ABS16, 
>          BFD_RELOC_CRX_ABS32, BFD_RELOC_CRX_NUM8, BFD_RELOC_CRX_NUM16, 
>          BFD_RELOC_CRX_NUM32, BFD_RELOC_CRX_IMM16 and 
BFD_RELOC_CRX_IMM32
>          * targets.c (bfd_elf32_crx_vec): Declare.
>          (bfd_target_vector): Add bfd_elf32_crx_vec.

> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
> 
>        * MAINTAINERS: Added myself to the list.
>          * readelf.c: include "elf/crx.h".
>          (guess_is_rela): Handle EM_CRX.
>          (get_machine_name): Ditto.
>          (dump_relocations): Ditto.

> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
> 
>          * Makefile.am (CPU_TYPES): Add crx.
>          (TARGET_CPU_CFILES): Add config/tc-crx.c.
>          (TARGET_CPU_HFILES): Add config/tc-crx.h.
>          (DEPTC_crx_elf): New target.
>          (DEPOBJ_crx_elf): Likewise.
>          (DEP_crx_elf): Likewise.
>          * Makefile.in: Regenerate.
>          * configure.in: Add crx* target.
>          * configure: Regenerate.
>          * config/tc-crx.c: New file.
>          * config/tc-crx.h: Likewise.

> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
>          * dis-asm.h (print_insn_crx): Declare.
> 
> include/elf/
>          * common.h (EM_CRX): Define.
>          * crx.h: New file.
> 
> include/opcode/
>          * crx.h: New file.
> 

> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
> 
>          * Makefile.am (ALL_EMULATIONS): Add eelf32crx.o.
>          (eelf32crx.c): New target.
>          * Makefile.in: Regenerate.
>          * configure.tgt: Handle crx-*-elf*.
>          * emulparams/elf32crx.sh: New file.
>          * emultempl/crxelf.em: Likewise.
>          * scripttempl/elf32crx.sc: Likewise.

> 2004-07-06  Tomer Levi  <Tomer.Levi@nsc.com>
> 
>          * Makefile.am (CFILES): Add crx-dis.c, crx-opc.c.
>          (ALL_MACHINES): Add crx-dis.lo, crx-opc.lo.
>          (crx-dis.lo): New target.
>          (crx-opc.lo): Likewise.
>          * Makefile.in: Regenerate.
>          * configure.in: Handle bfd_crx_arch.
>          * configure: Regenerate.
>          * crx-dis.c: New file.
>          * crx-opc.c: Likewise.
>          * disassemble.c (ARCH_crx): Define.
>          (disassembler): Handle ARCH_crx.

Approved and applied.