This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
RE: [PATCH 1/3] Enable Intel WAITPKG instructions.
>>> On 11.04.18 at 19:27, <igor.v.tsimbalist@intel.com> wrote:
>> -----Original Message-----
>> From: Jan Beulich [mailto:JBeulich@suse.com]
>> Sent: Monday, April 9, 2018 3:50 PM
>> To: Tsimbalist, Igor V <igor.v.tsimbalist@intel.com>
>> Cc: Lu, Hongjiu <hongjiu.lu@intel.com>; binutils@sourceware.org
>> Subject: Re: [PATCH 1/3] Enable Intel WAITPKG instructions.
>>
>> >>> On 09.04.18 at 15:41, <igor.v.tsimbalist@intel.com> wrote:
>> > Intel has disclosed a set of new instructions for Tremont processor.
>> > The spec is
>> > https://software.intel.com/en-us/intel-architecture-instruction-set-
>> extensions-pro
>> > gramming-reference
>> >
>> > This patch enables Intel WAITPKG instructions.
>> >
>> > gas/
>> > * config/tc-i386.c (cpu_arch): Add WAITPKG.
>> > (cpu_noarch): Likewise.
>> > * doc/c-i386.texi: Document WAITPKG.
>> > * i386/i386.exp: Run WAITPKG tests.
>> > * testsuite/gas/i386/waitpkg-intel.d: New test.
>> > * testsuite/gas/i386/waitpkg.d: Likewise.
>> > * testsuite/gas/i386/waitpkg.s: Likewise.
>> > * testsuite/gas/i386/x86-64-waitpkg-intel.d: Likewise.
>> > * testsuite/gas/i386/x86-64-waitpkg.d: Likewise.
>> > * testsuite/gas/i386/x86-64-waitpkg.s: Likewise.
>> >
>> > opcodes/
>> > * i386-dis.c (enum): Add PREFIX_MOD_0_0FAE_REG_6,
>> > PREFIX_MOD_1_0FAE_REG_6.
>> > * i386-dis-evex.h (prefix_table):
>> > New instructions (see prefixes above).
>> > * i386-gen.c (cpu_flag_init): Add WAITPKG.
>> > (cpu_flags): Likewise.
>> > * i386-opc.h (enum): Likewise.
>> > (i386_cpu_flags): Likewise.
>> > * i386-opc.tbl: Add umonitor, umwait, tpause.
>>
>> I don't think IgnoreSize is necessary anywhere here. While there
>> are many bad examples, I think it would be helpful if we stopped
>> adding unnecessary attributes to new insns. Quite far down on
>> my list of things to change is an item to drop all unnecessary
>> attributes, such that the table isn't any larger (and more difficult
>> to read/inspect) than needed.
>
> Updated patch is attached. Ok for trunk?
Thanks! Btw, the title says 1/3, but I've never seen 2/3 and 3/3
appear.
Jan