[PATCH 0/2]Enable Intel AVX512_FP16 instructions and add tests for it

[Cui, Lili]

> -----Original Message-----
> From: H.J. Lu <hjl.tools@gmail.com>
> Sent: Friday, July 2, 2021 1:47 AM
> To: Cui, Lili <lili.cui@intel.com>
> Cc: Binutils <binutils@sourceware.org>; Jan Beulich <jbeulich@suse.com>
> Subject: Re: [PATCH 0/2]Enable Intel AVX512_FP16 instructions and add tests
> for it
> 
> On Thu, Jul 1, 2021 at 8:42 AM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > On Thu, Jul 1, 2021 at 12:47 AM Cui,Lili <lili.cui@intel.com> wrote:
> > >
> > >
> > > Hi all,
> > >
> > > AVX512FP16 is disclosed, there're 100+ instructions for AVX512FP16,
> > > 2 patches for the convenience review, for details please refer to
> > >
> https://software.intel.com/content/www/us/en/develop/download/intel-
> > > avx512-fp16-architecture-specification.html
> > >
> > > Make check-gas is ok.
> > >
> > > Ok for master ?
> > >
> > > Cui,Lili (2):
> > >   Enable Intel AVX512_FP16 instructions
> > >   Add tests for Intel AVX512_FP16 instructions
> > >
> > >  gas/config/tc-i386.c                          |   15 +-
> > >  gas/doc/c-i386.texi                           |    4 +-
> > >  gas/testsuite/gas/i386/avx512_fp16-intel.d    | 1480 ++++++++++++++++
> > >  .../gas/i386/avx512_fp16-inval-bcast.l        |   27 +
> > >  .../gas/i386/avx512_fp16-inval-bcast.s        |   33 +
> > >  gas/testsuite/gas/i386/avx512_fp16.d          | 1480 ++++++++++++++++
> > >  gas/testsuite/gas/i386/avx512_fp16.s          | 1475 ++++++++++++++++
> > >  .../gas/i386/avx512_fp16_disp8-intel.d        |   20 +
> > >  gas/testsuite/gas/i386/avx512_fp16_disp8.d    |   20 +
> > >  gas/testsuite/gas/i386/avx512_fp16_disp8.s    |   16 +
> > >  gas/testsuite/gas/i386/avx512_fp16_vl-intel.d | 1438 +++++++++++++++
> > >  gas/testsuite/gas/i386/avx512_fp16_vl.d       | 1437 +++++++++++++++
> > >  gas/testsuite/gas/i386/avx512_fp16_vl.s       | 1433 +++++++++++++++
> > >  gas/testsuite/gas/i386/i386.exp               |   14 +
> > >  .../gas/i386/x86-64-avx512_fp16-intel.d       | 1544 +++++++++++++++++
> > >  .../gas/i386/x86-64-avx512_fp16-inval-bcast.l |   27 +
> > >  .../gas/i386/x86-64-avx512_fp16-inval-bcast.s |   33 +
> > >  gas/testsuite/gas/i386/x86-64-avx512_fp16.d   | 1544
> +++++++++++++++++
> > >  gas/testsuite/gas/i386/x86-64-avx512_fp16.s   | 1539 ++++++++++++++++
> > >  .../gas/i386/x86-64-avx512_fp16_disp8-intel.d |   20 +
> > >  .../gas/i386/x86-64-avx512_fp16_disp8.d       |   20 +
> > >  .../gas/i386/x86-64-avx512_fp16_disp8.s       |   16 +
> > >  .../gas/i386/x86-64-avx512_fp16_vl-intel.d    | 1438 +++++++++++++++
> > >  .../gas/i386/x86-64-avx512_fp16_vl.d          | 1438 +++++++++++++++
> > >  .../gas/i386/x86-64-avx512_fp16_vl.s          | 1433 +++++++++++++++
> > >  .../gas/i386/x86-64-default-suffix-avx.d      |    3 +
> > >  .../gas/i386/x86-64-default-suffix.d          |    3 +
> > >  .../gas/i386/x86-64-default-suffix.s          |    3 +
> > >  gas/testsuite/gas/i386/xmmword.l              |    2 +
> > >  gas/testsuite/gas/i386/xmmword.s              |    3 +
> > >  opcodes/i386-dis-evex-mod.h                   |   10 +
> > >  opcodes/i386-dis-evex-prefix.h                |  212 +++
> > >  opcodes/i386-dis-evex-w.h                     |  442 ++++-
> > >  opcodes/i386-dis-evex.h                       |  600 ++++++-
> > >  opcodes/i386-dis.c                            |  263 ++-
> > >  opcodes/i386-gen.c                            |    7 +-
> > >  opcodes/i386-opc.h                            |    7 +
> > >  opcodes/i386-opc.tbl                          |  376 ++++
> > >  38 files changed, 19843 insertions(+), 32 deletions(-)  create mode
> > > 100644 gas/testsuite/gas/i386/avx512_fp16-intel.d
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16-inval-bcast.l
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16-inval-bcast.s
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16.d
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16.s
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16_disp8-intel.d
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16_disp8.d
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16_disp8.s
> > >  create mode 100755 gas/testsuite/gas/i386/avx512_fp16_vl-intel.d
> > >  create mode 100755 gas/testsuite/gas/i386/avx512_fp16_vl.d
> > >  create mode 100644 gas/testsuite/gas/i386/avx512_fp16_vl.s
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16-intel.d
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16-inval-bcast.l
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16-inval-bcast.s
> > >  create mode 100644 gas/testsuite/gas/i386/x86-64-avx512_fp16.d
> > >  create mode 100644 gas/testsuite/gas/i386/x86-64-avx512_fp16.s
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16_disp8-intel.d
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16_disp8.d
> > >  create mode 100644
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16_disp8.s
> > >  create mode 100755
> > > gas/testsuite/gas/i386/x86-64-avx512_fp16_vl-intel.d
> > >  create mode 100755 gas/testsuite/gas/i386/x86-64-avx512_fp16_vl.d
> > >  create mode 100644 gas/testsuite/gas/i386/x86-64-avx512_fp16_vl.s
> > >
> > > --
> > > 2.17.1
> >
> > OK for both.
> >
> > Please give Jan a few days to comment.

Yes, the patch is big and it will take some time to review.

>
> Please add an entry to gas/NEWS for AVX512FP16 support.
> 

Ok, I will add it, thanks.


> 
> --
> H.J.