This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: drop {X,Y,Z}MMWORD_MNEM_SUFFIX
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Jan Beulich <JBeulich at suse dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Wed, 7 Mar 2018 05:43:28 -0800
- Subject: Re: [PATCH] x86: drop {X,Y,Z}MMWORD_MNEM_SUFFIX
- Authentication-results: sourceware.org; auth=none
- References: <5A9FF61702000078001AF636@prv-mh.provo.novell.com>
On Wed, Mar 7, 2018 at 5:24 AM, Jan Beulich <JBeulich@suse.com> wrote:
> They aren't really useful (anymore?): The conflicting operand size check
> isn't applicable to any insn validly using respective memory operand
> sizes (and if they're used wrongly, another error would result), and the
> logic in process_suffix() can be easily changed to work without them.
>
> While re-structuring conditionals in process_suffix() also drop the
> CMPXCHG8B special case in favor of a NoRex64 attribute in the opcode
> table.
>
> gas/
> 2018-03-07 Jan Beulich <jbeulich@suse.com>
>
> * config/tc-i386.c (XMMWORD_MNEM_SUFFIX, YMMWORD_MNEM_SUFFIX,
> ZMMWORD_MNEM_SUFFIX): Delete.
> (process_suffix): Drop their uses. Re-arrange final part of
> logic into a switch() statement. Drop special casing of
> cmpxchg8b.
>
> opcodes/
> 2018-03-07 Jan Beulich <jbeulich@suse.com>
>
> * i386-opc.tbl (cmpxchg8b): Add NoRex64.
> * i386-tlb.h: Re-generate.
>
OK.
Thanks.
--
H.J.