This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 4/4] x86: fold certain AVX and AVX2 templates
- From: "Jan Beulich" <JBeulich at suse dot com>
- To: "Alan Modra" <amodra at gmail dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>,<binutils at sourceware dot org>
- Date: Tue, 02 Jan 2018 03:49:45 -0700
- Subject: Re: [PATCH 4/4] x86: fold certain AVX and AVX2 templates
- Authentication-results: sourceware.org; auth=none
- References: <5A33A9D802000078001979BE@prv-mh.provo.novell.com> <5A33B37102000078001979FC@prv-mh.provo.novell.com> <20171222085425.GE22286@bubble.grove.modra.org>
>>> On 22.12.17 at 09:54, <amodra@gmail.com> wrote:
> On Fri, Dec 15, 2017 at 03:35:13AM -0700, Jan Beulich wrote:
>> * i386-gen.c (operand_type_init): Delete OPERAND_TYPE_REGYMM and
>> OPERAND_TYPE_REGZMM entries.
>
> This change should have resulted in a regen of opcodes/i386-init.h,
> and there are uses of the defines in gas/config/tc-i386.c (inside
> #ifdef DEBUG386 so you won't normally see a compilation error).
Indeed. I had realized the missing update to the generated header,
but had meant to update it with further changes to come hopefully
during the next weeks, as the leftover wouldn't have caused any
problems. But with the symbols actually used, I've now pushed a
partial revert. I'd like to note though that pt() as a whole doesn't
really look to be correct anymore; I've taken a note to look into
getting this to work right again.
Jan