public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Zack Weinberg <zack@codesourcery.com>
To: Nick Clifton <nickc@redhat.com>
Cc: binutils <binutils@sourceware.org>,
	 Alan Modra <amodra@bigpond.net.au>,
	Richard Earnshaw <rearnsha@arm.com>,
	Daniel Jacobowitz <dan@codesourcery.com>
Subject: Re: Second ping - Thumb-2 assembler
Date: Wed, 18 May 2005 06:23:00 -0000	[thread overview]
Message-ID: <87d5rp851x.fsf@codesourcery.com> (raw)
In-Reply-To: <4289EBB4.3020804@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 5181 bytes --]

Nick Clifton <nickc@redhat.com> writes:

> Hi Zack,
>
>> Last week I sent a query asking what still needed to be done before
>> the Thumb-2 assembler patches could be applied to binutils mainline:
>
>> but there has been no word since then.  How shall we proceed?
>
> Sorry - my bad - I have been swamped with work again.  Anyway - please
> proceed by checking in the entire patch in one go, as you
> proposed. Plus to answer your questions - no there are no objections
> to the design you have used and no there are no other(*) outstanding
> issues.

Thank you.  Attached (compressed) is the patch I have checked in.  I
was fairly specific about change logs outside of config/tc-arm.c, but
very brief about the changes there.

zw

include/elf:
        * arm.h: Import complete list of official relocation names
        and numbers from AAELF.  Define FAKE_RELOCs for old names.
        Remove a few old names no longer used anywhere.

bfd:
        * elf32-arm.c: Wherever possible, use official reloc names
        from AAELF.
        (elf32_arm_howto_table, elf32_arm_tls_gd32_howto)
        (elf32_arm_tls_ldo32_howto, elf32_arm_tls_ldm32_howto)
        (elf32_arm_tls_le32_howto, elf32_arm_tls_ie32_howto)
        (elf32_arm_vtinherit_howto, elf32_arm_vtentry_howto)
        (elf32_arm_pc11_howto, elf32_arm_thm_pc9_howto, elf32_arm_got_prel)
        (elf32_arm_r_howto): Replace with elf32_arm_howto_table_1,
        elf32_arm_howto_table_2, and elf32_arm_howto_table_3.
        Add many new relocations from AAELF.
        (elf32_arm_howto_from_type): Update to match.
        (elf32_arm_reloc_map): Add entries for R_ARM_THM_JUMP24,
        R_ARM_THM_JUMP11, R_ARM_THM_JUMP19, R_ARM_THM_JUMP8,
        R_ARM_THM_JUMP6, R_ARM_GNU_VTINHERIT, and R_ARM_GNU_VTENTRY.
        (elf32_arm_reloc_type_lookup): Use elf32_arm_howto_from_type.
        (elf32_arm_final_link_relocate): Add support for
        R_ARM_THM_JUMP24, R_ARM_THM_JUMP19, R_ARM_THM_JUMP6.  Remove
        case entries redundant with default.

        * reloc.c: Reorganize ARM relocations.  Add Thumb
        assembler-internal relocations BFD_RELOC_ARM_T32_OFFSET_U8,
        BFD_RELOC_ARM_T32_OFFSET_IMM, BFD_RELOC_ARM_T32_IMMEDIATE.
        Add visible relocations BFD_RELOC_THUMB_PCREL_BRANCH7,
        BFD_RELOC_THUMB_BRANCH20, BFD_RELOC_THUMB_BRANCH25.
        Delete unused relocations BFD_RELOC_ARM_GOT12, BFD_RELOC_ARM_COPY.
        * bfd-in2.h, libbfd.h: Regenerate.

opcodes:

        * arm-dis.c (thumb_opcodes): Add disassembly for V6T2 16-bit
        instructions.  Adjust disassembly of some opcodes to match
        unified syntax.
        (thumb32_opcodes): New table.
        (print_insn_thumb): Rename print_insn_thumb16; don't handle
        two-halfword branches here.
        (print_insn_thumb32): New function.
        (print_insn): Choose among print_insn_arm, print_insn_thumb16,
        and print_insn_thumb32.  Be consistent about order of
        halfwords when printing 32-bit instructions.

gas:
        * hash.c (hash_lookup): Add len parameter.  All callers changed.
        (hash_find_n): New interface.
        * hash.h: Prototype hash_find_n.
        * sb.c: Include as.h.
        (scrub_from_sb, sb_to_scrub, scrub_position): New statics.
        (sb_scrub_and_add_sb): New interface.
        * sb.h: Prototype sb_scrub_and_add_sb.
        * input-scrub.c (input_scrub_include_sb): Use sb_scrub_and_add_sb.

        * config/tc-arm.h (TC_FORCE_RELOCATION_LOCAL): Remove
        reference to BFD_RELOC_ARM_GOT12 which is never generated.
        * config/tc-arm.c: Rewrite, adding Thumb-2 support.

gas/testsuite:
        * gas/arm/arm.exp: Convert all existing "gas_test" tests to
        "run_dump_test" tests.  Run more tests unconditionally.  Run new tests.
        * gas/arm/arch4t.s, gas/arm/arch6zk.s, gas/arm/arm3.s, gas/arm/arm6.s
        * gas/arm/arm7dm.s, gas/arm/bignum1.s, gas/arm/float.s
        * gas/arm/immed.s, gas/arm/iwmmxt.s, gas/arm/offset.s, gas/arm/thumb.s:
        Adjust to work as a dump test.
        * gas/arm/arch4t.d, gas/arm/arch6zk.d, gas/arm/arm3.d, gas/arm/arm6.d
        * gas/arm/arm7dm.d, gas/arm/bignum1.d, gas/arm/float.d
        * gas/arm/immed.d, gas/arm/iwmmxt.d, gas/arm/offset.d, gas/arm/thumb.d:
        New files.

        * gas/arm/armv1-bad.l, gas/arm/armv1-bad.s: Remove tests for
        diagnostics that don't happen in the first pass anymore.

        * gas/arm/iwmmxt-bad.l, gas/arm/r15-bad.l, gas/arm/req.l
        * gas/arm/vfp-bad.l:
        Update expected diagnostics.
        * gas/arm/pic.d: Update expected reloc name.
        * gas/arm/thumbv6.d: CPY no longer appears in disassembly.
        * gas/arm/r15-bad.s: Avoid two-argument mul.
        * gas/arm/req.s: Adjust comments.
        * gas/arm/maverick.d, gas/arm/maverick.s: Avoid inappropriate
        use of PC.

        * gas/arm/macro-1.d, gas/arm/macro1.s
        * gas/arm/t16-bad.l, gas/arm/t16-bad.s
        * gas/arm/tcompat.d, gas/arm/tcompat.s
        * gas/arm/tcompat2.d, gas/arm/tcompat2.s
        * gas/arm/thumb32.d, gas/arm/thumb32.s
        New test pair.

ld/testsuite:
        * ld-arm/mixed-app.d: Adjust expected disassembly a little.


[-- Attachment #2: thumb-2.diff.gz --]
[-- Type: application/gzip, Size: 209176 bytes --]

      reply	other threads:[~2005-05-18  5:41 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-11 20:58 Zack Weinberg
2005-05-17 13:05 ` Nick Clifton
2005-05-18  6:23   ` Zack Weinberg [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87d5rp851x.fsf@codesourcery.com \
    --to=zack@codesourcery.com \
    --cc=amodra@bigpond.net.au \
    --cc=binutils@sourceware.org \
    --cc=dan@codesourcery.com \
    --cc=nickc@redhat.com \
    --cc=rearnsha@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).