This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [RFA/ARM] Fix latent bugs in parse_neon_mov

From: Matthew Gretton-Dann <matthew dot gretton-dann at arm dot com>
To: "binutils at sourceware dot org" <binutils at sourceware dot org>
Cc: Richard Earnshaw <Richard dot Earnshaw at arm dot com>
Date: Wed, 07 Dec 2011 13:55:03 +0000
Subject: Re: [RFA/ARM] Fix latent bugs in parse_neon_mov
References: <4EDF6C79.1040806@arm.com>

This time with a non-empty patch :-).

Matt

On 07/12/11 13:39, Matthew Gretton-Dann wrote:

This patch fixes some latent bugs in parse_neon_mov in gas/config/tc-arm.c.
On exit from parse_neon_mov *which_operand should reflect the index of
the last operand parsed, and not the next operand to parse.  This was
not done in all cases, and the attached patch fixes this.
This issue was identified whilst preparing a patch (forthcoming) to stop
gas mis-accepting: vcvt.f64.s32 d0, s0, #11.
Tested on arm-none-eabi.

OK?

gas/ChangeLog:

2011-12-06 Matthew Gretton-Dann<matthew.gretton-dann@arm.com>
	* config/tc-arm.c (parse_neon_mov): Update which_operand
	correctly.


--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltd

Attachment: 1-PATCH-ARM-Fix-latent-bugs-in-parse_neon_mov.txt
Description: Text document

Follow-Ups:
- Re: [RFA/ARM] Fix latent bugs in parse_neon_mov
  - From: Richard Earnshaw

References:
- [RFA/ARM] Fix latent bugs in parse_neon_mov
  - From: Matthew Gretton-Dann

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]