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: [PATCH, AArch64] Change not to accept AL or NV condition in CINC, CSET, CINV, CSETM and CNEG


On 5 November 2013 17:54, Yufeng Zhang <Yufeng.Zhang@arm.com> wrote:
> Hi,
>
> This patch updates the AArch64 gas and binutils as a response to a recent
> ISA update.  The aliases CINC, CSET, CINV, CSETM, and CNEG require that the
> assembler and disassembler invert bit 0 of the standard "cond" encoding to
> achieve the expected behavior. However when the specified condition is AL
> (cond=='1110') this will not deliver the intended result, since the
> condition code named NV (cond=='1111') behaves identically to AL, i.e.
> always true.
>
> For these aliases gas now should no longer accept the AL or NV conditions,
> and the preferred disassembly for the CSINC, CSINV or CSNEG instructions
> when cond=='111x' shall be the instructions themselves.
>
> OK for the trunk and 2.24?

OK /Marcus


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