This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86/Intel: accept mandated operand order for vcvt{,u}si2s{d,s}
- From: "Jan Beulich" <JBeulich at suse dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: "Kirill Yukhin" <kirill dot yukhin at gmail dot com>, "Binutils" <binutils at sourceware dot org>,"H. Peter Anvin" <hpa at zytor dot com>
- Date: Wed, 06 May 2015 08:44:34 +0100
- Subject: Re: [PATCH] x86/Intel: accept mandated operand order for vcvt{,u}si2s{d,s}
- Authentication-results: sourceware.org; auth=none
- References: <552FE0630200007800072CD0 at mail dot emea dot novell dot com> <CAMe9rOomWLMwQT6R2qLr1p7_dzmwuNLsz2PEk-6tV3NTt_=24Q at mail dot gmail dot com> <55390A6A0200007800075263 at mail dot emea dot novell dot com> <CAMe9rOrSX3cgXCSJFsbDnKgnyNrX-Y4cYO2VL4uTrKV-O3fcCw at mail dot gmail dot com> <554906C70200007800076D28 at mail dot emea dot novell dot com> <CAMe9rOp2bUnsU+Tn4RKLNw48h6JGiaGaX1ddqpGhyM3tsF7pTg at mail dot gmail dot com>
>>> On 05.05.15 at 18:10, <hjl.tools@gmail.com> wrote:
> On Tue, May 5, 2015 at 9:07 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>> On 23.04.15 at 15:17, <hjl.tools@gmail.com> wrote:
>>> On Thu, Apr 23, 2015 at 6:06 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>>>> On 23.04.15 at 14:39, <hjl.tools@gmail.com> wrote:
>>>>> It is not OK.
>>>>
>>>> ... I guess as the Intel syntax maintainer I could decide to ignore
>>>> this.
>>>
>>> MASM AVX512 compatibility isn't our goal. Compatible with NASM is
>>> a good ideal. Peter, Kirill, let's work it out.
>>>
>>> Adding Peter for NASM and Kirill for GAS.
>>
>> Not having seen any response from them at all, I think applying
>> at least the assembler side (which leaves the current bogus
>> operand order available) should really not be controversial. As
>> to the disassembler side, I continue to think that Intel syntax
>> disassembly should preferably match the Intel manual, especially
>> when there is no other implementation to use as reference.
>>
>> Thoughts?
>>
>
> Since there is no MASM AVX512 compatibility to speak of,
> please don't apply those patches
Please don't just repeat yourself, but give a reason I can understand
to override the intention to conform with the Intel manual. I'm
certainly hesitant to commit changes that can't be agreed upon, but
as said before I don't feel tied to your disapproval of the changes.
Jan