This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [binutils][gas][arm] Remove bogus entry added by accident in former patch
- From: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>
- To: "Andre Vieira (lists)" <Andre dot SimoesDiasVieira at arm dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Fri, 9 Jun 2017 11:25:21 +0100
- Subject: Re: [binutils][gas][arm] Remove bogus entry added by accident in former patch
- Authentication-results: sourceware.org; auth=none
- References: <593A746D.8070609@arm.com>
On 09/06/17 11:11, Andre Vieira (lists) wrote:
> Hi,
>
> Thanks to Richard for noticing this! In an old patch
> (https://sourceware.org/ml/binutils/2015-08/msg00028.html) an entry for
> a non-existing encoding of bx was added, I have no idea where this came
> from and it is obviously wrong and unintended. My apologies for this.
> This patch removes that entry.
>
> Best Regards,
> Andre
>
> 2017-07-09 Andre Vieira <andre.simoesdiasvieira@arm.com>
>
> * arm-dis.c (print_insn_arm): Remove bogus entry.
>
>
> diff
>
>
> diff --git a/opcodes/arm-dis.c b/opcodes/arm-dis.c
> index a0a03ee1f87fa87ebeb85aa12e3f1f6b711e5838..fbb7c10a4e2b623c86aa1548be1b1ec1135245d6 100644
> --- a/opcodes/arm-dis.c
> +++ b/opcodes/arm-dis.c
> @@ -2310,8 +2310,6 @@ static const struct opcode32 arm_opcodes[] =
> 0x01300000, 0x0ff00010, "teq%p%c\t%16-19r, %o"},
> {ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
> 0x01300010, 0x0ff00010, "teq%p%c\t%16-19R, %o"},
> - {ARM_FEATURE_CORE_LOW (ARM_EXT_V5),
> - 0x0130f000, 0x0ff0f010, "bx%c\t%0-3r"},
>
> {ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
> 0x03400000, 0x0fe00000, "cmp%p%c\t%16-19r, %o"},
>
OK.