This is the mail archive of the
binutils@sourceware.cygnus.com
mailing list for the binutils project.
Re: CPSR changes to ARM binutils in the mainline...
- To: Scott Bambrough <scottb at netwinder dot org>
- Subject: Re: CPSR changes to ARM binutils in the mainline...
- From: Philip Blundell <Philip dot Blundell at pobox dot com>
- Date: Sat, 27 May 2000 13:09:50 +0100
- cc: Nick Clifton <nickc at cygnus dot com>, binutils mailing list <binutils at sourceware dot cygnus dot com>
- References: <392E7CC5.C4D29BA8@netwinder.org>
>The mainline is correct. The ARM Architecture Reference states in note 1 that
>the immediate form can only be used to set the flag bits. The results are
>unpredictable with the other fields, although it seems to work. However, I
>think your patch should go into the branch for consistency. I don't think it
>is a good idea for the branch and the mainline to have different behaviour here,
>particularly when we are just about to release.
>
>What do you think?
My preference (for both branch and trunk) would actually be to issue a warning
rather than an error for these sorts of instructions. This is more consistent
with how the assembler handles other syntactically correct but invalid
instructions.
Anyway, applying the patch to the branch is OK so long as there is mention in
NEWS of the change. The previous release accepted these instructions so the
change needs to be documented.
p.