This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: fix (dis)assembly of certain SSE2 insns in 16-bit mode
- 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: Mon, 24 Jun 2019 08:57:46 -0700
- Subject: Re: [PATCH] x86: fix (dis)assembly of certain SSE2 insns in 16-bit mode
- References: <5D10C9E5020000780023AA1E@prv1-mh.provo.novell.com>
On Mon, Jun 24, 2019 at 6:02 AM Jan Beulich <JBeulich@suse.com> wrote:
>
> MOVNTI was wrongly assembled with a 66h prefix. Add IgnoreSize to
> address this. It and the scalar to/from integer conversion insns also
> were also wrongly using Ev / Gv, leading to 16-bit register names being
> printed when 32-bit ones were meant.
>
> Clone the 32-bit SSE2 test to cover both assembler and disassembler.
>
> gas/
> 2019-06-24 Jan Beulich <jbeulich@suse.com>
>
> * testsuite/gas/i386/sse2-16bit.d,
> testsuite/gas/i386/sse2-16bit.s: New.
> testsuite/gas/i386/i386.exp: Run new test.
>
> opcodes/
> 2019-06-24 Jan Beulich <jbeulich@suse.com>
>
> * i386-dis.c (prefix_table): Use Edq for cvtsi2ss and cvtsi2sd.
> Use Gdq for cvttss2si, cvttsd2si, cvtss2si, and cvtsd2si, and
> movnti.
> * i386-opc.tbl (movnti): Add IgnoreSize.
> * i386-tbl.h: Re-generate.
>
OK.
Thanks.
--
H.J.