This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 4/5] x86: convert broadcast insn attribute to boolean
- 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: Thu, 22 Mar 2018 03:26:48 -0700
- Subject: Re: [PATCH 4/5] x86: convert broadcast insn attribute to boolean
- References: <5AB267180200008903664A6E@prv-mh.provo.novell.com> <5AB2789902000078001B47C7@prv-mh.provo.novell.com> <5AB27D3A02000078001B480D@prv-mh.provo.novell.com>
On Wed, Mar 21, 2018 at 7:41 AM, Jan Beulich <JBeulich@suse.com> wrote:
> [re-sending with the opcodes/i386-opc.tbl changes removed from the
> mail body, instead adding a compressed attachment of the full patch]
>
> The (only) valid broadcast type for an insn can be inferred from other
> information.
>
> gas/
> 2018-03-21 Jan Beulich <jbeulich@suse.com>
>
> * config/tc-i386.c (struct Broadcast_Operation): Adjust comment.
> (check_VecOperands): Re-write broadcast validation code.
> (check_VecOperations): Replace BROADCAST_1TO* uses.
> * testsuite/gas/i386/inval-avx512f.s: Add various broadcast
> cases.
> * testsuite/gas/i386/inval-avx512f.l: Adjust expectations.
>
> opcodes/
> 2018-03-21 Jan Beulich <jbeulich@suse.com>
>
> * i386-opc.h (NO_BROADCAST, BROADCAST_1TO16, BROADCAST_1TO8,
> BROADCAST_1TO4, BROADCAST_1TO2): Delete.
> (struct i386_opcode_modifier): Shrink broadcast field to 1 bit.
> * i386-opc.tbl: Replace Broadcast=<N> by Broadcast.
> * i386-tlb.h: Re-generate.
>
OK.
Thanks.
--
H.J.