This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH v2] x86-64: don't allow use of %axl as accumulator
>>> On 14.11.17 at 13:37, <hjl.tools@gmail.com> wrote:
> On Tue, Nov 14, 2017 at 12:17 AM, Jan Beulich <JBeulich@suse.com> wrote:
>> --- 2017-11-10/gas/testsuite/gas/i386/x86-64-reg-bad.s
>> +++ 2017-11-10/gas/testsuite/gas/i386/x86-64-reg-bad.s
>> @@ -0,0 +1,18 @@
>> +# Check %axl / %cxl aren't permitted as accumulator / shift count, but
>> +# make sure any of the extended low registers still work as intended
>> + .text
>> +reg:
>> + movabs -1, %axl
>> + add $1, %axl
>> + add $1, %cxl
>> + add $1, %dxl
>> + add $1, %bxl
>> + add $1, %spl
>> + add $1, %bpl
>> + add $1, %sil
>> + add $1, %dil
>
> Please move
>
> + add $1, %axl
> + add $1, %cxl
> + add $1, %dxl
> + add $1, %bxl
> + add $1, %spl
> + add $1, %bpl
> + add $1, %sil
> + add $1, %dil
>
> to x86-64-specific-reg.s. OK with that change.
>
> Thanks for your patience.
And it was not possible to tell me right away where you want this
added, just to make me go through one more round? H.J, please!
Also, please trim your replies.
Jan